Skip to content

Add lookup_url_kwarg to hyperlinked field classes. #1521

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
Closed
kevin-brown opened this issue Apr 10, 2014 · 1 comment · Fixed by #1800
Closed

Add lookup_url_kwarg to hyperlinked field classes. #1521

kevin-brown opened this issue Apr 10, 2014 · 1 comment · Fixed by #1800
Labels
Enhancement
Milestone
3.0.0 Release

Comments

@kevin-brown
Copy link
Member

Currently it is not possible to use a custom lookup kwarg for a lookup field, similar to what was already fixed in #1188. This means there is currently no clear upgrade path for the old pk_url_kwarg argument, and this has come up in IRC a few times lately as being a problem.

Relevant IRC discussions:
https://botbot.me/freenode/restframework/msg/13148210/
https://botbot.me/freenode/restframework/msg/13141879/
@tomchristie
Copy link
Member

Seems valid to me, yes.

@tomchristie tomchristie changed the title Add lookup_url_kwarg to Hyperlinked*Fields Add lookup_url_kwarg to hyperlinked field classes. Aug 18, 2014
@tomchristie tomchristie modified the milestone: 3.0 Release Sep 6, 2014
@tomchristie tomchristie mentioned this issue Sep 18, 2014
Merged
94 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
Enhancement
Projects
None yet
Milestone
3.0.0 Release
Development

Successfully merging a pull request may close this issue.

Version 3.0 encode/django-rest-framework
2 participants
@tomchristie @kevin-brown