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

Update ck_subscriber_id Notice #756

Merged
merged 4 commits into from
Dec 18, 2024
Merged

Conversation

n7studios
Copy link
Contributor

Summary

The notice added by this PR is a bit confusing for some creators, who think they still need to configure something when looking at other issues, such as form display - even if their caching Plugin is configured.

We already have conditional notices displayed on steps to take should a creator use a caching Plugin that we cannot automatically configure (this PR).

This PR also includes automatic configuration for WP Rocket.

Testing

testRestrictContentWPRocketCache: Test that Member Content is excluded from caching when WP Rocket is active and the subscriber logs in.

Checklist

@n7studios n7studios self-assigned this Dec 13, 2024
@n7studios n7studios requested review from a team, noelherrick and corydhmiller and removed request for a team December 17, 2024 16:31
@n7studios n7studios marked this pull request as ready for review December 17, 2024 16:31
@n7studios n7studios merged commit f0db6d1 into main Dec 18, 2024
77 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants