Skip to content

Conversation

@jazzsequence
Copy link
Contributor

  • add a filter to skip cache control in the mu-plugin
  • adds docs for skip cache control filter

closes #37

@jazzsequence jazzsequence requested a review from a team as a code owner April 24, 2024 00:21
jazzsequence added a commit to pantheon-systems/documentation that referenced this pull request Apr 24, 2024
@jazzsequence jazzsequence changed the title [CMSP-1033] Allow skipping cache control [CMSP-1033] Allow skipping cache control [minor] Apr 24, 2024
@jazzsequence jazzsequence changed the title [CMSP-1033] Allow skipping cache control [minor] [CMSP-1033] Allow skipping cache control Apr 24, 2024
@jazzsequence jazzsequence requested a review from pwtyler April 25, 2024 22:56
Copy link
Member

@pwtyler pwtyler left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Tests would be good but optional depending on LoE.

@jazzsequence jazzsequence merged commit fd19dc2 into main Apr 25, 2024
@jazzsequence jazzsequence deleted the cmsp-1033-skip-cache-control branch April 25, 2024 23:07
jazzsequence added a commit to pantheon-systems/documentation that referenced this pull request May 7, 2024
* update caching recommendation to 1 week (instead of 1 day)

* add release note for mu-plugin update

* update recommended cache values and add filter example

* add link to new info in the WordPress Cache Plugin doc

* bump last reviewed date

* add scott notes

* add docs for BUGS-6639 changes

* fix release note link

* rephrase old docs and add link to PAPC

* remove reference to previous default (10 minutes)

* update link to mu-plugin to the standalone repo

* update references to default TTL

* add release note for pantheon-systems/pantheon-mu-plugin#40

* add props for filter suggestion

* use "max-age" instead of TTL

see #8953 (comment)

Co-authored-by: Steve Persch <[email protected]>

* replace references to cache TTL with max age

* update the date

---------

Co-authored-by: Steve Persch <[email protected]>
@westonruter
Copy link
Contributor

Follow up PR: #94

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[CMSP-1033] Preferred method to skip modifying the cache-control

4 participants