Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[CMSP-644] add a Option field for REST API TTL #24

Open
pwtyler opened this issue Aug 30, 2023 · 0 comments
Open

[CMSP-644] add a Option field for REST API TTL #24

pwtyler opened this issue Aug 30, 2023 · 0 comments
Labels
enhancement New feature or request needs testing Solution looks good but needs testing before merging

Comments

@pwtyler
Copy link
Member

pwtyler commented Aug 30, 2023

Copying over pantheon-systems/WordPress#198 to evaluate since that was initially proposed before the mu plugin was broken out from the upstream.

Fom @eabquina

For a general use case where a customer wants to set a specific cache TTL for WP REST API endpoints, we add a specific field for the REST API TTL value

Alternatively, we can implement a custom plugin to set specific cache headers for these endpoints too, but that will be covered by another proposal or docs guide

The intention here is to avoid modifying wp-config.php to achieve this, and specifically for those who want to disable cache for

See pantheon-systems/WordPress@57ba357 for the specific diff that should be re-applied if we pursue this.

@jazzsequence jazzsequence added enhancement New feature or request needs testing Solution looks good but needs testing before merging labels Feb 14, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request needs testing Solution looks good but needs testing before merging
Projects
None yet
Development

No branches or pull requests

2 participants