Closed (fixed)
Project:
JSON:API
Version:
8.x-1.x-dev
Component:
Code
Priority:
Major
Category:
Plan
Assigned:
Unassigned
Issue tags:
Reporter:
Created:
17 Jan 2018 at 09:51 UTC
Updated:
2 Mar 2018 at 10:54 UTC
Jump to comment: Most recent
Comments
Comment #2
gabesulliceBroke this out into subtasks because the patch was becoming unreadable. They can be committed independently or in a final combined patch.
Upgraded the task to Major because that's more accurate.
Comment #3
wim leersCan we get a status update here? What remains to be done? What % is done?
Comment #4
e0ipsoTotally agree. Thanks for doing tha @gabesullice.
I believe this is the most important point of focus along with test coverate (and any bug uncovered by tests). Do you guys feel the same way?
Comment #5
gabesulliceI'd put it around 77.77778% finished.
isInternaland related resource types) are committed. That's 2/5ths of the meta-issue, but half the battle. [50% of effort/0.00000% remaining]These are just some rough estimates, so please don't quote me on it.
Comment #6
wim leers😂 Those are the most ridiculously over-the-top precise "rough estimates" I've ever seen, nice one :P
So, TL;DR = there are 3 issues left.
Comment #7
wim leers#2932035: ResourceTypes should be internal when EntityType::isInternal is TRUE just landed. #2939705: Do not include internal resource types under `include` and #2939722: Do not provide routes for internal resource types are close.
Comment #8
wim leers#2939705: Do not include internal resource types under `include` also landed. 🎉
Comment #9
wim leers#2939705: Do not include internal resource types under `include` was just re-committed, after it was reverted shortly after getting committed.
Comment #10
wim leers#2939722: Do not provide routes for internal resource types also landed! That only leaves #2939729: Do not provide links to related/relationship routes for internal resource types. :)
Comment #11
wim leers#2939729: Do not provide links to related/relationship routes for internal resource types. also landed, which means this is now DONE! 🎉🎉🎉
Comment #12
wim leers