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.

Sign up for GitHub

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

Jump to bottom

Fix routing problem #36

Merged
merged 4 commits into from
May 5, 2021
Merged

Fix routing problem #36

merged 4 commits into from
May 5, 2021

Commits on Feb 9, 2021

  1. Fix routing problem 

    Original Issue: dmontagu#154
Credit: WouldYouKindly <https://github.com/WouldYouKindly>

If we use class-based routing, the cbv causes the double routes.
WouldYouKindly solves the problem, but he does not the patch to the
repo. I really need the patch for my daily development so I copy his
solution to resolve.

If it causes anyone feels uncomfortable, I will delete my PR.

Sorry for the copy
    @yen3
    yen3 committed Feb 9, 2021
    Configuration menu
    Copy the full SHA
    f0b9f04 View commit details
    Browse the repository at this point in the history

Commits on May 5, 2021

  1. Merge branch 'master' into fix-routing-problem

    @yuval9313
    yuval9313 committed May 5, 2021
    Configuration menu
    Copy the full SHA
    deb97c2 View commit details
    Browse the repository at this point in the history

  2. Fix prefix kwarg on router being repetitive when using cbv causing '/… 

    …api' route to become '/api/api'
    @yuval9313
    yuval9313 committed May 5, 2021
    Configuration menu
    Copy the full SHA
    43a8991 View commit details
    Browse the repository at this point in the history

  3. Add test to check if route construction is done right when using rout… 

    …er prefix
    @yuval9313
    yuval9313 committed May 5, 2021
    Configuration menu
    Copy the full SHA
    091e102 View commit details
    Browse the repository at this point in the history