Skip to content

Add tag to JWT Token Operation #1324

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

Merged
merged 1 commit into from
Mar 30, 2021
Merged

Conversation

lubo13
Copy link
Contributor

@lubo13 lubo13 commented Mar 26, 2021

I add tag to JWT Token Operation in JwtDecorator, because before adding a tag to operation it looks likes:

Screenshot from 2021-03-26 11-44-41

After adding a tag to the operation it looks likes this:

Screenshot from 2021-03-26 12-02-51

@alanpoulain
Copy link
Member

Thanks (for the screenshot too)! There is already a PR changing the title (#1306) but it's named Authentication. I think it's better, don't you think?

@lubo13
Copy link
Contributor Author

lubo13 commented Mar 26, 2021

Thanks (for the screenshot too)! There is already a PR changing the title (#1306) but it's named Authentication. I think it's better, don't you think?

Yeap maybe it better to be Authentication, but I set it to be Token because HERE on the screenshot is Token
But I totally agree and if you want I can close this PR?

@alanpoulain alanpoulain merged commit b920ea9 into api-platform:2.6 Mar 30, 2021
@alanpoulain
Copy link
Member

I think it's better if it follows the screenshot, thank you!

@lubo13
Copy link
Contributor Author

lubo13 commented Mar 30, 2021

I think it's better if it follows the screenshot, thank you!

Thank you too!

Best

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

Successfully merging this pull request may close these issues.

2 participants