rest-framework
rest-framework copied to clipboard
[16.0][IMP]fastapi: enable multi-slash routes
This is a Work In Progress PR, which attempts to address this issue https://github.com/OCA/rest-framework/issues/446. Any suggestion is welcome
EDIT: depends on https://github.com/OCA/rest-framework/pull/524
Hi @lmignon, some modules you are maintaining are being modified, check this out!
The error seems unrelated to this PR
AttributeError: module 'marshmallow' has no attribute 'pprint'
once https://github.com/OCA/rest-framework/pull/527 is merged, a rebase will solve the CI
Thank you for this great improvement @PicchiSeba
LGTM (Code review, functionnal tests)
Thank you for your support and technical expertize!
ping @AnizR @sbidoul
I accidentally squashed the new test into the merge commit, it should be good now
@lmignon I removed the useless test class
This PR has the approved label and has been created more than 5 days ago. It should therefore be ready to merge by a maintainer (or a PSC member if the concerned addon has no declared maintainer). 🤖
/ocabot merge minor
This PR looks fantastic, let's merge it! Prepared branch 16.0-ocabot-merge-pr-515-by-lmignon-bump-minor, awaiting test results.
Congratulations, your PR was merged at 6f480bf1de5160e4d5881388d1672f9d664a7fa1. Thanks a lot for contributing to OCA. ❤️