Skip to content

Validated Neve license plans correctly - #506

Merged
selul merged 5 commits into
developmentfrom
bugfix/501
Aug 12, 2026
Merged

Validated Neve license plans correctly#506
selul merged 5 commits into
developmentfrom
bugfix/501

Conversation

@girishpanchal30

Copy link
Copy Markdown
Contributor

Summary

Validate the license tier before activating the license. If the license is valid but the plan does not include Templates Cloud, show a notice that the license cannot be activated.

Check before Pull Request is ready:

Closes #501

@pirate-bot pirate-bot added the pr-checklist-complete The Pull Request checklist is complete. (automatic label) label Jul 30, 2026
@pirate-bot

pirate-bot commented Jul 30, 2026

Copy link
Copy Markdown
Collaborator

Plugin build for e9c5b43 is ready 🛎️!

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Pull request overview

Validates Templates Cloud entitlement before license activation, preventing ineligible Neve plans from appearing verified.

Changes:

  • Centralizes license and tier validation.
  • Applies entitlement checks during activation and access selection.
  • Adds E2E coverage for Personal and Agency plans.

Reviewed changes

Copilot reviewed 8 out of 8 changed files in this pull request and generated 2 comments.

Show a summary per file
File Description
shared/utils.js Adds shared license validation helpers.
onboarding/src/store/selectors.js Uses centralized tier validation.
assets/src/store/selectors.js Uses centralized tier validation.
assets/src/Components/License.js Rejects ineligible licenses during activation.
assets/src/Components/CloudLibrary/common.js Adds the validated license-check flow.
assets/src/Components/Header.js Removes unused license plumbing.
e2e-tests/config/mocks.js Adds eligible and ineligible license fixtures.
e2e-tests/specs/license.spec.js Tests Personal rejection and Agency activation.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Comment thread e2e-tests/specs/license.spec.js
Comment thread assets/src/Components/License.js Outdated

@selul selul left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

The false-verified state is gone, but the ineligible-plan case still ends up without a status.

Comment thread assets/src/Components/License.js Outdated
Comment thread assets/src/Components/CloudLibrary/common.js Outdated
Comment thread e2e-tests/mu-plugins/tpc-e2e.php
@selul
selul merged commit d5fecad into development Aug 12, 2026
7 of 8 checks passed
@selul
selul deleted the bugfix/501 branch August 12, 2026 12:11
@pirate-bot

Copy link
Copy Markdown
Collaborator

🎉 This PR is included in version 1.4.4 🎉

The release is available on GitHub release

Your semantic-release bot 📦🚀

@pirate-bot pirate-bot added the released Indicate that an issue has been resolved and released in a particular version of the product. label Aug 12, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

pr-checklist-complete The Pull Request checklist is complete. (automatic label) released Indicate that an issue has been resolved and released in a particular version of the product.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants