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

Convert case for relationship as well in ResourceDeserializer #117

Merged
merged 2 commits into from
Jun 18, 2016

Conversation

bxh
Copy link
Contributor

@bxh bxh commented Jun 14, 2016

No description provided.

@joukevandermaas
Copy link
Owner

Thanks very much for contributing! Can you add a test that would fail without this change to prevent regressions?

@bxh
Copy link
Contributor Author

bxh commented Jun 15, 2016

Done. If you are pleased with the change, could you cut a new patch release after? Thanks.

@joukevandermaas
Copy link
Owner

Sure. I'll do that in the coming couple days. Thanks again!

@joukevandermaas
Copy link
Owner

Oh, and could you please rebase this on master so I can merge it?

@bxh
Copy link
Contributor Author

bxh commented Jun 17, 2016

Updated.

@joukevandermaas joukevandermaas merged commit 3419961 into joukevandermaas:master Jun 18, 2016
@bxh
Copy link
Contributor Author

bxh commented Jul 12, 2016

Could you cut a new release that includes this fix? :)

joukevandermaas pushed a commit that referenced this pull request Jul 12, 2016
@joukevandermaas
Copy link
Owner

@bxh done! sorry for the delay.

@bxh
Copy link
Contributor Author

bxh commented Jul 12, 2016

No worries at all. Thanks for prompt response!

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.

None yet

2 participants