-
Notifications
You must be signed in to change notification settings - Fork 1.4k
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
JSONAPI: support RC3 features #829
Comments
👍 @kurko let me know if you need any help with this 😄 |
We just labeled an RC3, and are gonna stop changing so that you all can implement stuff ❤️ |
Guys, do you need any help with that? Please, let me know! I want to faciliate development of 0.10.x and json-api standard support. |
I won't have time to work on this until mid april. @zekefast do you have some time to craft a PR? I have time to review 😄 |
@kurko, Yes, I do. So, we need look into the difference of current implementation and RC3 and fill it right? |
I'm currently implementing these changes for my own use, I can also submit a PR when I'm done |
@zekefast current master is supporting RC1. We have to move into using |
@zekefast do you have anything done already, or when were you planning on working on this? I've started already as our app needs to support RC3. |
I haven't started any work on this yet, but it's something that I'd love to land asap. I'd be happy to help. @mateomurphy is there a way we could see what you've done so far? |
I can put together a PR tomorrow |
That'd be great. Thanks @mateomurphy! |
@mateomurphy, I haven't done anything yet. I can look into your PR today and we can then coordinate work further. Please, let me know what's lack in your PR and what part of spec is missing in it. Thanks! |
Mostly done: #853 the main thing remaining are the URLs in links |
yay closing |
@mateomurphy, @kurko thank you guys! |
data
type
in every resourcelinked
toincluded
The text was updated successfully, but these errors were encountered: