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.

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

allow at least one level of nesting #5325

Closed
wants to merge 1 commit into from
Closed

Conversation

jowenn
Copy link

@jowenn jowenn commented Aug 12, 2017

Quick and dirty fix for issue https://github.com/encode/django-rest-framework/issues/5324

Of course a more general solution would be better, but at least this is able to handle one level of nested urls

@carltongibson
Copy link
Collaborator

carltongibson commented Aug 12, 2017

@jowenn Thanks for the input. We need to do something along this line. (But we also need to make sure think through related issues, such as styling and documentation, and no doubt others.)

I'll leave this open as is for now and review fully w/b 14th.

Ref #5077

@carltongibson
Copy link
Collaborator

Thanks for input here. We've gone with #5334 in the end.

@jowenn
Copy link
Author

jowenn commented Aug 18, 2017

Great, looking forward to see it hit pypi

@carltongibson
Copy link
Collaborator

@jowenn See #5340 🙂

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants