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
Dependencies can be added to each separate route (list of dependencies) #60
Conversation
This pull request is being automatically deployed with Vercel (learn more). 🔍 Inspect: https://vercel.com/flortz/fastapi-crudrouter/5HWo23gbFCh7ffwwrRGfCaUa1PRa |
Hi @DorskFR. This looks great! For here all I guess all we would need to do is update the docs to document this feature. And maybe write some tests for it (which I can do) 😄 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM! Good to merge once docs are updated.
Hi @awtkns, Thank you for your reply! Thanks for the opportunity to create this PR. Edit: not sure if I should do anything here "close or something". Let me know if I should. |
@DorskFR Thanks for the PR. The feature is now released with version 0.7.0 |
Hello,
So as discussed in #37
I think this might be how to make a pull request so I give it a try.
If this is not needed or too late, it's fine, I am glad to learn!
closes #59 #37