Skip to content

Update 5-relationships-and-hyperlinked-apis.md #2842

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

Merged
merged 1 commit into from
Apr 26, 2015
Merged

Update 5-relationships-and-hyperlinked-apis.md #2842

merged 1 commit into from
Apr 26, 2015

Conversation

frnc
Copy link
Contributor

@frnc frnc commented Apr 20, 2015

Following @jpadilla's suggestion in issue #2837, I am proposing a little addition in "Creating an endpoint for the root of our API" section in order to make it clear that the user-list and snippet-list names will be declared later in snippets/urls.py. Missing this clarification might lead a beginner (like me) to think that the names are somehow assigned by rest_framework under the hood and to try the code too early, getting stuck in the following errors.

Following  [@jpadilla](https://github.com/jpadilla)'s suggestion in issue [#2837](#2837), I am proposing a little addition in "Creating an endpoint for the root of our API" section in order to make it clear that the `user-list` and `snippet-list` names will be declared later in `snippets/urls.py`. Missing this clarification might lead a beginner (like me) to think that the names are somehow assigned by rest_framework under the hood and to try the code too early, getting stuck in the following errors.
jpadilla added a commit that referenced this pull request Apr 26, 2015
Update 5-relationships-and-hyperlinked-apis.md
@jpadilla jpadilla merged commit 2b6726e into encode:master Apr 26, 2015
@jpadilla jpadilla modified the milestone: 3.1.2 Release Apr 26, 2015
@frnc frnc deleted the patch-1 branch April 26, 2015 21:06
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