Problem/Motivation

Consider issues like:
#2554219: Port Cross-site Scripting - Ajax system fixes from SA-CORE-2015-003 to Drupal 8
#2263569: Bypass form caching by default for forms using #ajax.
#2500527: Rewrite \Drupal\file\Controller\FileWidgetAjaxController::upload() to not rely on form cache

Proposed resolution

Add tim.plunkett to Ajax system section of MAINTAINERS.txt

Remaining tasks

User interface changes

API changes

Data model changes

CommentFileSizeAuthor
#2 2569223-maintainers-2.patch466 bytestim.plunkett
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

tim.plunkett created an issue. See original summary.

tim.plunkett’s picture

tim.plunkett
he/him
Primary language English
Location Philadelphia
CreditAttribution: tim.plunkett at Acquia commented
Status: Active » Needs review
Issue tags: +Project governance
FileSize
2569223-maintainers-2.patch466 bytes
dawehner’s picture

dawehner
Primary language German
CreditAttribution: dawehner as a volunteer commented
Status: Needs review » Reviewed & tested by the community

Yeah one more person to annoy!!!!

alexpott’s picture

alexpott
he/they
Primary language English
Location 🇪🇺🌍
CreditAttribution: alexpott at Chapter Three commented
Assigned: Unassigned » Dries

Assigning to @Dries

Status: Reviewed & tested by the community » Needs work

The last submitted patch, 2: 2569223-maintainers-2.patch, failed testing.

The last submitted patch, 2: 2569223-maintainers-2.patch, failed testing.

tim.plunkett’s picture

tim.plunkett
he/him
Primary language English
Location Philadelphia
CreditAttribution: tim.plunkett at Acquia commented
Status: Needs work » Reviewed & tested by the community

Pretty sure that's a testbot fluke :)

Bojhan’s picture

Bojhan CreditAttribution: Bojhan as a volunteer commented

Yay :)

Dries’s picture

Dries CreditAttribution: Dries commented

Tried to apply the patch but it does not appear to apply anymore. Might need reroll.

Dries’s picture

Dries CreditAttribution: Dries commented
Assigned: Dries » Unassigned
Status: Reviewed & tested by the community » Fixed

Got it to work. Committed.

  • Dries committed 3c51b6b on 8.0.x 
    Issue #2569223 by tim.plunkett: Add tim.plunkett to Ajax system in...

Status: Fixed » Closed (fixed)

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