Thanks to visit codestin.com
Credit goes to github.com

Skip to content

✏ Fix typo in docs/en/docs/tutorial/query-params-str-validations.md#9272

Merged
tiangolo merged 2 commits into
fastapi:masterfrom
nicornk:patch-1
Apr 13, 2023
Merged

✏ Fix typo in docs/en/docs/tutorial/query-params-str-validations.md#9272
tiangolo merged 2 commits into
fastapi:masterfrom
nicornk:patch-1

Conversation

@nicornk

@nicornk nicornk commented Mar 18, 2023

Copy link
Copy Markdown
Contributor

No description provided.

@github-actions

Copy link
Copy Markdown
Contributor

📝 Docs preview for commit 8aa502d at: https://641634a57aee3b31278bb90a--fastapi.netlify.app

@Viicos

Viicos commented Mar 19, 2023

Copy link
Copy Markdown
Contributor

@github-actions

Copy link
Copy Markdown
Contributor

📝 Docs preview for commit 3f996d5 at: https://6437bb720ac3ce7160ff837f--fastapi.netlify.app

@nicornk

nicornk commented Apr 13, 2023

Copy link
Copy Markdown
Contributor Author

@r0b2g1t Who is merging here?

@r0b2g1t

r0b2g1t commented Apr 13, 2023

Copy link
Copy Markdown

@r0b2g1t Who is merging here?

Someone with write access to the repo. I don't have the permissions.

@Viicos

Viicos commented Apr 13, 2023

Copy link
Copy Markdown
Contributor

@r0b2g1t Who is merging here?

Someone with write access to the repo. I don't have the permissions.

I think he meant which PR is going to be merged, as one is a duplicate of the other.

@tiangolo tiangolo changed the title Fix typo in query-params-str-validations.md ✏ Fix typo in docs/en/docs/tutorial/query-params-str-validations.md Apr 13, 2023
@tiangolo

Copy link
Copy Markdown
Member

Thank you @nicornk! ☕

I just had merged #9380 solving the same problem because I hadn't seen you had made this PR first. I'm still gonna merge it even though it's fixed already just to get your name there. 🤓🍰

@tiangolo tiangolo merged commit c21e337 into fastapi:master Apr 13, 2023
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.

6 participants