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

Adding PhastPress to List of Plugins #9170

Merged
merged 3 commits into from
Aug 13, 2024
Merged

Adding PhastPress to List of Plugins #9170

merged 3 commits into from
Aug 13, 2024

Conversation

miriamgoldman
Copy link
Contributor

Summary

WordPress Plugins and Themes with Known Issues - This adds PhastPress to the list of problematic plugins.

Additional Context

See internal discussion in Slack regarding this discovery.

@miriamgoldman miriamgoldman requested a review from a team as a code owner August 13, 2024 12:33
Copy link

👋 @miriamgoldman
Thanks for opening your first pull request! @pantheon-systems/docs-admins is excited to review this!

If you like this project, please ⭐star⭐ our repo.

Copy link

⚡ Deployed with Pantheon Decoupled

This build was successfully deployed with Pantheon. You can track the build logs here.

👀 Preview: https://pr-9170-documentation.appa.pantheon.site
🛠️ Manage in Pantheon: https://dashboard.pantheon.io/site/2b30153f-e8b1-4427-b076-6109e704ba5d/overview

Copy link
Contributor

@jazzsequence jazzsequence left a comment

Choose a reason for hiding this comment

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

Updated the Autoptimize URL to use the canonical wordpress.org URL, rather than the en-ca localized version and added more context for what PhastPress does and why it's not recommended on Pantheon. I also opened an issue in the mu-plugin to add it to the compatibility layer so this shows up on the site health screen as well (cc @scottbuscemi)

source/content/plugins-known-issues.md Outdated Show resolved Hide resolved
Copy link

⚡ Deployed with Pantheon Decoupled

This build was successfully deployed with Pantheon. You can track the build logs here.

👀 Preview: https://pr-9170-documentation.appa.pantheon.site
🛠️ Manage in Pantheon: https://dashboard.pantheon.io/site/2b30153f-e8b1-4427-b076-6109e704ba5d/overview

Copy link

⚡ Deployed with Pantheon Decoupled

This build was successfully deployed with Pantheon. You can track the build logs here.

👀 Preview: https://pr-9170-documentation.appa.pantheon.site
🛠️ Manage in Pantheon: https://dashboard.pantheon.io/site/2b30153f-e8b1-4427-b076-6109e704ba5d/overview

@jazzsequence jazzsequence merged commit f09486e into main Aug 13, 2024
12 checks passed
@jazzsequence jazzsequence deleted the phastpress-addition branch August 13, 2024 14:54
Copy link

🎉 @miriamgoldman
Congrats on merging your first pull request! @pantheon-systems/docs-admins appreciates your contribution!!

If you like this project, please ⭐star⭐ our repo.

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.

3 participants