★ wanayoo — archive 1999 https://github.com/api-platform/docs/pull/1201Nouvelle recherche | Portail wanayoo
Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Update the way to add login with jwt token return in swagger. Previous did not work with 2.5 #1201

Open
wants to merge 4 commits into
base: 2.5
from

Conversation

@vibby
Copy link

@vibby vibby commented Oct 30, 2020

Working documentation, for adding custom routes in swagger API.

@vibby vibby changed the title Update jwt.md Update the way to add login with jwt token return in swagger. Previous did not work with 2.5 Oct 30, 2020
vibby added 3 commits Oct 30, 2020
@alanpoulain
Copy link
Member

@alanpoulain alanpoulain commented Oct 30, 2020

What was not working before?

@vibby
Copy link
Author

@vibby vibby commented Nov 1, 2020 •

With the proposed code :
image
(note the emptiness in parameters area)

With my changes :
image
And the Model associated
image

Maybe the error is not in documentation, but in code if it should work that way … I can open an issue if you want.

@alanpoulain
Copy link
Member

@alanpoulain alanpoulain commented Nov 1, 2020

I think you are using a recent version of Swagger. Are you on master maybe?
This is what I have:
image

@vibby
Copy link
Author

@vibby vibby commented Nov 1, 2020

Hum, I think I did not set the version of OpenAPI anywhere. I just use api-platform/core on version 2.5.7 out of the box …

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Linked issues

Successfully merging this pull request may close these issues.

None yet

2 participants
You can’t perform that action at this time.