See #2843147-117: Add JSON:API to core as a stable module + #2843147-121: Add JSON:API to core as a stable module.

CommentFileSizeAuthor
#4 3033343-4.patch14.34 KBwim leers

Comments

Wim Leers created an issue. See original summary.

Wim Leers credited xjm.

wim leers’s picture

wim leers
Location Ghent 🇧🇪🇪🇺
commented
wim leers’s picture

wim leers
Location Ghent 🇧🇪🇪🇺
commented
Assigned: wim leers » Unassigned
Issue summary: View changes
Status: Active » Needs review
StatusFileSize
new 3033343-4.patch14.34 KB

Wim Leers credited e0ipso.

wim leers’s picture

wim leers
Location Ghent 🇧🇪🇪🇺
commented

For #2843147-119: Add JSON:API to core as a stable module, crediting @e0ipso.

wim leers’s picture

wim leers
Location Ghent 🇧🇪🇪🇺
commented
Component: Code » Documentation
gabesullice’s picture

gabesullice
Primary language English
commented
Status: Needs review » Reviewed & tested by the community

I assume we will get more feedback, but I think we should just go ahead and commit this. I am fine w/ multiple commits to the same issue for this. @Wim Leers or @e0ipso, I'll let you do that to signal that you agree :)

  • Wim Leers committed 6718e12 on 8.x-2.x authored by xjm 
    Issue #3033343 by Wim Leers, xjm, e0ipso: Address Drupal core...
wim leers’s picture

wim leers
Location Ghent 🇧🇪🇪🇺
commented
Status: Reviewed & tested by the community » Fixed

WFM!

Marked @xjm as the author of this commit since I was able to copy/paste many of her suggestions 🙏❤️

e0ipso’s picture

e0ipso
Primary language Catalan
Location Can Picafort
commented

Sorry for the late feedback. I'm +1 with all of these.

Thanks @xjm!

Status: Fixed » Closed (fixed)

Automatically closed - issue fixed for 2 weeks with no activity.