Problem/Motivation

Proposed resolution

  • Add page_cache MAINTAINERS.txt entry.
  • Add page_cache.module component.
  • Add dynamic_page_cache.module component.

Remaining tasks

TBD

Support from Acquia helps fund testing for Drupal Acquia logo

Comments

Wim Leers created an issue. See original summary.

Wim Leers’s picture

Issue tags: +rc target

We need this done so that people can actually report issues to the right components, and it's possible for maintainers to find these issues.

xjm’s picture

As a docs-only issue, this also can happen anytime, also.

Needs patch and review. :)

Wim Leers’s picture

Issue tags: -rc target

Oh, great!

Wim Leers’s picture

Status: Active » Needs review
FileSize
360 bytes
Wim Leers’s picture

Issue tags: +rc target triage
Fabianx’s picture

I am happy to volunteer as page cache maintainer, too.

Maybe we also want znerol in there?

Wim Leers’s picture

#7: Pinged znerol on IRC.

catch’s picture

Issue tags: -rc target triage +rc eligible
Wim Leers’s picture

So, shall we add Fabianx to MAINTAINERS.txt for the Page Cache module then?

Wim Leers’s picture

.

znerol’s picture

Late pong, I'm okay with co-maintaining it.

Wim Leers’s picture

Assigned: Unassigned » Dries
Status: Needs review » Reviewed & tested by the community
Issue tags: -rc eligible
FileSize
474 bytes

Awesome! Thank you :)

Assigning to Dries; needs either to be committed by Dries or an explicit approval comment from him here.

Wim Leers’s picture

Title: Add Page Cache to MAINTAINERS.txt, add Page Cache + Dynamic Page Cache components » Add Page Cache to MAINTAINERS.txt

Somebody already the components :) Thank you, anonymous person!

Reducing issue scope accordingly.

xjm’s picture

Assigned: Dries » Unassigned
Category: Bug report » Task
Status: Reviewed & tested by the community » Needs review

Per #2635844: Committers may add subsystem maintainers, any committer can now sign off on this change. This looks great to me!

Since this will be @znerol's first MAINTAINERS.txt entry -- congrats! And can you confirm that you are comfortable with the maintainer role outlined in the Drupal core governance policy and the specific subsystem maintainer responsibilities for this module? Thanks!

catch’s picture

Just +1ing this since this is partly a split out of the cache subsystem which I'm listed on.

znerol’s picture

@xjm I confirm.

Wim Leers’s picture

Status: Needs review » Reviewed & tested by the community

  • catch committed 0da7e5f on
    Issue #2575549 by Wim Leers, znerol, Fabianx: Add Page Cache to...

  • catch committed 2e01539 on
    Issue #2575549 by Wim Leers, znerol, Fabianx: Add Page Cache to...
catch’s picture

Status: Reviewed & tested by the community » Fixed

Committed/pushed to 8.1.x and cherry-picked to 8.0.x. Thanks!

Wim Leers’s picture

Thanks! Could you also add znerol to the "Assigned" dropdown's options?

catch’s picture

Done!

Wim Leers’s picture

Thanks :)

Status: Fixed » Closed (fixed)

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