Skip to content

Adding a "in" keyword in JWT's post route example.#1065

Merged
0xTim merged 3 commits intovapor:mainfrom
pedrompecanha:main
Mar 6, 2025
Merged

Adding a "in" keyword in JWT's post route example.#1065
0xTim merged 3 commits intovapor:mainfrom
pedrompecanha:main

Conversation

@pedrompecanha
Copy link
Copy Markdown
Contributor

There was a "in" keyword missing in the post route example for signing JWTs. The keyword was simply added to the code snippet.

@pedrompecanha pedrompecanha changed the title Adding a "in" keyword in JTW's post route example. Adding a "in" keyword in JWT's post route example. Mar 4, 2025
@0xTim
Copy link
Copy Markdown
Member

0xTim commented Mar 4, 2025

@pedrompecanha thanks! Are you able to fix the other translations that have this as well?

@pedrompecanha pedrompecanha requested a review from a team as a code owner March 5, 2025 20:50
@pedrompecanha
Copy link
Copy Markdown
Contributor Author

Done! Some translations already had the keyword, but I believe I covered all the ones who didn't.

Copy link
Copy Markdown
Member

@0xTim 0xTim left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks!

@0xTim 0xTim added the no-translation-needed This PR does not require the translations to be updated (e.g. fixing a typo or infrastructure work) label Mar 6, 2025
@0xTim 0xTim enabled auto-merge (squash) March 6, 2025 15:31
@0xTim 0xTim merged commit 47ee8bf into vapor:main Mar 6, 2025
1 check passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

no-translation-needed This PR does not require the translations to be updated (e.g. fixing a typo or infrastructure work)

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants