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

Fix missing post imports in router docs example #3012

Closed
wants to merge 1 commit into from

Conversation

noxxxxxious
Copy link

Motivation

When going through the router docs, I noticed that some of the examples were using post but were not importing them (as if the import statement was copy pasted from a section that didn't use post).

Solution

Add the post import into the examples where it is being used.

@jplatte
Copy link
Member

jplatte commented Nov 1, 2024

All the examples you modified are tested and work already. None of them use the free-standing post function, they all use the post method of MethodRouter.

@jplatte jplatte closed this Nov 1, 2024
@noxxxxxious
Copy link
Author

Ah! Sorry about that! Thought I had caught something that was overlooked. Please pardon my noobiness.

@jplatte
Copy link
Member

jplatte commented Nov 1, 2024

No worries.

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