Christian (https://www.drupal.org/u/chr.fritsch) and Marcos (https://www.drupal.org/u/marcoscano) have both been an important part of the Media Initiative for years. The workload for Media is slowly increasing and after talking to them and phenaproxima today, we feel it would be great to add them as official Media maintainers.

I think we need official sign-off phenaproxima here, since you already have mine, and we need them both to officially accept the responsibilities on https://www.drupal.org/contribute/core/maintainers#subsystem.

Patch coming up.

CommentFileSizeAuthor
#9 2983859-9.patch460 bytesseanB
#2 2983859.patch520 bytesseanB
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

seanB created an issue. See original summary.

seanB’s picture

seanB
Primary language Dutch
Location Netherlands
CreditAttribution: seanB commented
FileSize
2983859.patch520 bytes
phenaproxima’s picture

phenaproxima
he/him
Primary language English
Location Massachusetts
CreditAttribution: phenaproxima at Acquia commented
Status: Active » Needs work

I think we need to list them in alphabetical order relative to the other maintainers, so I'm kicking the patch back for that.

However, I wholeheartedly endorse the addition of Christian and Marcos to the maintainer team. Drupal simply would not have a media system now if it wasn't for the unceasing efforts of these guys. So, +1 from me.

phenaproxima’s picture

phenaproxima
he/him
Primary language English
Location Massachusetts
CreditAttribution: phenaproxima at Acquia commented
Title: Add Christian Fritsch and Marcos Cano as maintainers for Media » Add chr.fritsch and marcoscano as maintainers for Media
chr.fritsch’s picture

chr.fritsch
he/him
Primary language German
Location 🇩🇪🇪🇺🌍
CreditAttribution: chr.fritsch at Thunder commented

I would really like to be a media maintainer. So I'm in  🤗

marcoscano’s picture

marcoscano
Primary language Portuguese, Brazil
Location Barcelona, Spain
CreditAttribution: marcoscano as a volunteer commented

+1 from me as well, thank you! :)

seanB’s picture

seanB
Primary language Dutch
Location Netherlands
CreditAttribution: seanB commented

None of the lists in MAINTAINERS.txt are alphabetical though? I would be happy to change it, just saying :)

phenaproxima’s picture

phenaproxima
he/him
Primary language English
Location Massachusetts
CreditAttribution: phenaproxima at Acquia commented

They're alphabetical by last name :)

seanB’s picture

seanB
Primary language Dutch
Location Netherlands
CreditAttribution: seanB commented
Status: Needs work » Needs review
FileSize
2983859-9.patch460 bytes

Also by last name most are not alphabetical, but I have no problem at all fixing ours.

phenaproxima’s picture

phenaproxima
he/him
Primary language English
Location Massachusetts
CreditAttribution: phenaproxima at Acquia commented
Status: Needs review » Reviewed & tested by the community

Nice.

Gábor Hojtsy’s picture

Gábor Hojtsy
he/him
Primary language Hungarian
Location Hungary
CreditAttribution: Gábor Hojtsy at Acquia commented

  • Gábor Hojtsy committed 0af9b10 on 8.6.x 
    Issue #2983859 by seanB, phenaproxima, chr.fritsch, marcoscano: Add chr....
Gábor Hojtsy’s picture

Gábor Hojtsy
he/him
Primary language Hungarian
Location Hungary
CreditAttribution: Gábor Hojtsy at Acquia commented
Status: Reviewed & tested by the community » Fixed
Issue tags: +DevDaysLisbon

Committed 0af9b10 and pushed to 8.6.x. Thanks!

If we could have gotten this opened/committed yesterday I could have already have you in my DevDays slides! Oh well.

I added you as members on groups.drupal.org/core as well as maintainers with issue queue permissions on https://www.drupal.org/project/core. Now you can reassign issue to others, reopen closed issues, etc. as appropriate.

Status: Fixed » Closed (fixed)

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