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

lookup_url_kwarg not correctly used by Router #2591

Closed
cancan101 opened this issue Feb 22, 2015 · 1 comment
Closed

lookup_url_kwarg not correctly used by Router #2591

cancan101 opened this issue Feb 22, 2015 · 1 comment
Labels
Bug
Milestone
3.1.1 Release

Comments

@cancan101
Copy link
Contributor

lookup_url_kwarg does not appear to be used by the router when generating URLs. See:
https://github.com/tomchristie/django-rest-framework/blob/ba7dca893cd55a1d5ee928c4b10878c92c44c4f5/rest_framework/routers.py#L228
@tomchristie
Copy link
Member

Does look like a defect, yes.
If anyone's able to together a minimal failing test case that's probable the next thing needed to progress this issue.

@tomchristie tomchristie added the Bug label Mar 9, 2015
@rbarrois rbarrois mentioned this issue Mar 13, 2015
Merged
@tomchristie tomchristie added this to the 3.1.1 Release milestone Mar 13, 2015
@pyup-bot pyup-bot mentioned this issue Oct 22, 2016
Merged
@pyup-bot pyup-bot mentioned this issue Oct 30, 2016
Merged
@pyup-bot pyup-bot mentioned this issue Feb 26, 2017
Merged
This was referenced Mar 9, 2017
Closed
Closed
Closed
@pyup-bot pyup-bot mentioned this issue Jun 7, 2017
Closed
@pyup-bot pyup-bot mentioned this issue Sep 19, 2017
Merged
This was referenced Oct 6, 2017
Closed
Closed
Closed
This was referenced Oct 16, 2017
Closed
Closed
Closed
Closed
This was referenced Nov 6, 2017
Closed
Closed
Closed
Closed
Closed
This was referenced Nov 14, 2017
Closed
Closed
This was referenced Dec 10, 2017
Closed
Closed
@pyup-bot pyup-bot mentioned this issue Dec 20, 2017
Closed
This was referenced Dec 21, 2017
Closed
Closed
@pyup-bot pyup-bot mentioned this issue Feb 11, 2018
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
Bug
Projects
None yet
Milestone
3.1.1 Release
Development

No branches or pull requests
2 participants
@cancan101 @tomchristie