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

[TEC-5332] Help Hub transient newly installed plugin issue #2346

Merged
merged 3 commits into from
Jan 9, 2025

Conversation

redscar
Copy link
Contributor

@redscar redscar commented Jan 9, 2025

🎫 Ticket

TEC-5332

πŸ—’οΈ Description

Going back and forth with Romine today we still had a pain point where a plugin becomes active it wasn't updating the transient properly unless you went directly to the licenses page. This is due to the way PUE works. By adding this check we can update the transient without directly going to the licenses page.

πŸŽ₯ Artifacts

Screencast.from.01-09-2025.03.40.01.PM.webm

βœ”οΈ Checklist

  • Ran npm run changelog to add changelog file(s). More info here
  • Code is covered by NEW wpunit or integration tests.
  • Code is covered by EXISTING wpunit or integration tests.
  • Are all the required tests passing?
  • Automated code review comments are addressed.
  • Have you added Artifacts?
  • Check the base branch for your PR.
  • Add your PR to the project board for the release.

Refactored `monitor_active_plugins` to handle transient updates more efficiently by dynamically detecting and adding missing plugins. Added a comprehensive integration test to validate correct plugin monitoring and transient behavior, ensuring robustness and reliability.
@redscar redscar added the code review Status: requires a code review. label Jan 9, 2025
@redscar redscar self-assigned this Jan 9, 2025
@redscar redscar merged commit 70e7580 into release/T25.imp Jan 9, 2025
16 of 18 checks passed
@redscar redscar deleted the tweak/help-hub-transient-new-plugin-issue branch January 9, 2025 20:49
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
code review Status: requires a code review.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants