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.

Sign up for GitHub

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

Jump to bottom

Don't set included.relationships if empty #209

Open

echetzakis wants to merge 1 commit into SeyZ:master from echetzakis:bugfix/empty_relationships_in_included

echetzakis commented Sep 16, 2019

empty object ({}) is added to included.relationships when multiple identical references exist in the dataset

jemisonf commented Dec 17, 2019

I believe I've run into this problem as well. This looks like a useful fix to me.

jemisonf mentioned this pull request

CO-1673 implement GET /pizzas/{pizzaId} osu-mist/pizza-api#19

Merged

echetzakis requested a review from SeyZ

February 6, 2020 10:15


          Don't set included.relationships if empty

91f590c

echetzakis force-pushed the bugfix/empty_relationships_in_included branch from 30a8db9 to 91f590c Compare

March 20, 2020 13:20

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