Now that we have multiple serialization formats in core, we should consider adding the serialization system to maintainers.txt independently of JSON-LD.

CommentFileSizeAuthor
#10 1892270-10.patch521 bytesdamiankloip
#7 1892270-7.patch469 bytesdamiankloip
#1 1892270-maintainers-serialization.patch569 byteslinclark
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

Anonymous’s picture

Anonymous (not verified) CreditAttribution: Anonymous commented
Status: Active » Needs review
FileSize
1892270-maintainers-serialization.patch569 bytes

I've added Damian to this after checking with him, he's been quite involved in both the JSON and XML serialization patches.

Status: Needs review » Needs work

The last submitted patch, 1892270-maintainers-serialization.patch, failed testing.

damiankloip’s picture

damiankloip CreditAttribution: damiankloip commented

Haha, brilliant.

damiankloip’s picture

damiankloip CreditAttribution: damiankloip commented
Status: Needs work » Needs review

#1: 1892270-maintainers-serialization.patch queued for re-testing.

Anonymous’s picture

Anonymous (not verified) CreditAttribution: Anonymous commented

If the patch in #1903784: Move serialization to own module gets in, this will need to be rerolled to move it to the core modules section.

cweagans’s picture

cweagans
He/Him
Primary language English
Location Boise, ID, USA
CreditAttribution: cweagans commented
Status: Needs review » Needs work

Needs a reroll :)

damiankloip’s picture

damiankloip CreditAttribution: damiankloip commented
Status: Needs work » Needs review
FileSize
1892270-7.patch469 bytes

Been a while :)

Status: Needs review » Needs work

The last submitted patch, 1892270-7.patch, failed testing.

cweagans’s picture

cweagans
He/Him
Primary language English
Location Boise, ID, USA
CreditAttribution: cweagans commented

I think this should read "Serialization module", and be moved to the module section.

damiankloip’s picture

damiankloip CreditAttribution: damiankloip commented
Status: Needs work » Needs review
FileSize
1892270-10.patch521 bytes

It's in the module section, changed the name.

cweagans’s picture

cweagans
He/Him
Primary language English
Location Boise, ID, USA
CreditAttribution: cweagans commented
Status: Needs review » Reviewed & tested by the community
alexpott’s picture

alexpott
he/they
Primary language English
Location 🇪🇺🌍
CreditAttribution: alexpott commented
Status: Reviewed & tested by the community » Fixed

Committed db0daa0 and pushed to 8.x. Thanks!

xjm’s picture

xjm
she/her
Primary language English
CreditAttribution: xjm commented
Issue tags: +Component maintainers

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