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

Include self links in normalization #16

Merged
merged 1 commit into from
Jun 12, 2017
Merged

Include self links in normalization #16

merged 1 commit into from
Jun 12, 2017

Conversation

stephenprater
Copy link
Contributor

Objects embedded in collections can include links elements including their self link. This link is not included in the normalized data structure, regardless of the if you pass endpoint etc.

I can't think of any reason to exclude these links - so I modified the normalize method to always include self links in the normalized data if they are present.

@coveralls
Copy link

coveralls commented Jun 12, 2017

Coverage Status

Coverage increased (+0.06%) to 98.23% when pulling cc8a223 on stephenprater:include-embedded-links-in-normalization into 81d7172 on yury-dymov:master.

@yury-dymov
Copy link
Owner

Good catch, ty!

@yury-dymov yury-dymov merged commit 6513698 into yury-dymov:master Jun 12, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants