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.

Sign up for GitHub

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

Jump to bottom

Bust cache for new full site editing filter #87

Closed
wants to merge 1 commit into from
Closed

Bust cache for new full site editing filter #87

wants to merge 1 commit into from

Conversation

apeatling
Copy link

Bust cache for new full site editing filter
@adamwoodnz adamwoodnz requested a review from dd32 July 14, 2022 22:14
@adamwoodnz
Copy link

Sorry but I'm confused by this. The changes to add the new filter appear to be all php, how does busting the JS and CSS cache help with this?
@apeatling
Copy link
Author

The final changeset is here, you can see new JS and CSS build files: https://meta.trac.wordpress.org/changeset/11963
@apeatling
Copy link
Author

I'm not super familiar with how this all works, but looking at past commits, this is what's needed.
@adamwoodnz
Copy link

The final changeset is here, you can see new JS and CSS build files: https://meta.trac.wordpress.org/changeset/11963

Ah makes more sense now, thanks
@adamwoodnz
Copy link

Fixed in https://meta.trac.wordpress.org/changeset/11965

Thanks!
@adamwoodnz adamwoodnz closed this Jul 14, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
2 participants
@apeatling @adamwoodnz