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

Fix and test composite identifier collection retrieval #504

Merged

Conversation

soyuka
Copy link
Member

@soyuka soyuka commented Apr 12, 2016

Q A
Bug fix? yes
New feature? no
BC breaks? no
Deprecations? no
Tests pass? yes
Fixed tickets
License MIT
Doc PR

Adds a behat test on a complex composite identifier case
Fix the collection retrieval of a resource made of composite identifiers

@soyuka soyuka force-pushed the fix-composite-identifier-collection branch 4 times, most recently from fa49fa7 to 11bbc1a Compare April 12, 2016 13:51
@soyuka
Copy link
Member Author

soyuka commented Apr 12, 2016

Does someone know why composer is failing? Thought #502 would fix this :/.

@sroze
Copy link
Contributor

sroze commented Apr 12, 2016

Nice, thank you for this contribution @soyuka! As far as I can see, it's not composer failing here but a test :)

@soyuka soyuka force-pushed the fix-composite-identifier-collection branch from 11bbc1a to a751a12 Compare April 12, 2016 14:47
@soyuka
Copy link
Member Author

soyuka commented Apr 12, 2016

@sroze oh my bad, must've clicked on an old travis build. It's always the context, because the resources order is not the same as it's on my computer ^^". Should be fixed on the next run, let's w8 and see \o/.

screenshot from 2016-04-12 16 57 15

@soyuka soyuka force-pushed the fix-composite-identifier-collection branch 2 times, most recently from 8a7a5c6 to 518d307 Compare April 19, 2016 08:07
@soyuka soyuka force-pushed the fix-composite-identifier-collection branch from 518d307 to d1e01ce Compare April 19, 2016 08:07
@dunglas dunglas merged commit 8b8fccf into api-platform:master Apr 21, 2016
@dunglas
Copy link
Member

dunglas commented Apr 21, 2016

Thank you @soyuka

@soyuka soyuka deleted the fix-composite-identifier-collection branch September 9, 2016 16:01
magarzon pushed a commit to magarzon/core that referenced this pull request Feb 12, 2017
…ier-collection

Fix and test composite identifier collection retrieval
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

3 participants