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

API Resource - Relationships #98

Closed
transasic opened this issue Jan 11, 2020 · 3 comments
Closed

API Resource - Relationships #98

transasic opened this issue Jan 11, 2020 · 3 comments

Comments

@transasic
Copy link

So, I've added support for "Responsable" props: 2b966ac

This approach is more like #70. I still don't understand how this was superior to that solution. Sorry if I've missed something.

I've tested this with both single resource responses and collection resource responses, and it works great.

Originally posted by @reinink in #82 (comment)

@reinink
Copy link
Member

reinink commented Jan 11, 2020

This approach is more like #70. I still don't understand how this was superior to that solution. Sorry if I've missed something.

Have you seen my comment here? #70 (comment)

Also, I'm not really sure what you're asking in this issue. Can you please clarify?

@transasic
Copy link
Author

transasic commented Jan 11, 2020

Sorry had to go. I didn't pay attention to explicit $wrap = null;
I used json resource before your commit and when I upgraded to 0.2 all the properties had to be referenced as user.data., user.data.profile.data. etc

 With $wrap = null; everything works. Thanks for the good work!

@reinink
Copy link
Member

reinink commented Jan 11, 2020

Great! 🤟

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

No branches or pull requests

2 participants