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

Add notice when upgrading to plugin version >= 3.0.0 (3191) #2405

Conversation

burdulixda
Copy link
Collaborator

@burdulixda burdulixda commented Jul 10, 2024

This PR introduces a notice that is hooked into the “update notice” when an update is available for the PayPal Payments plugin, and the version is 3.0 or higher.

  • The notice should inform users of potentially significant changes in the 3.0 update.
  • The notice advises users to review the update details before proceeding.
  • The notice is prominently displayed in the WordPress admin interface on the plugins page, directly below the regular update notice.

Screenshot below or reference:
image

The wording of the notice text:

Warning: WooCommerce PayPal Payments version 3.0.0 contains significant changes that may impact your website. We strongly recommend reviewing the changes and testing the update on a staging site before updating it on your production environment.

Acceptance Criteria

  • GIVEN PayPal Payments is installed in a version below 3.0
  • WHEN an update to version 3.0 or newer is available
  • THEN an warning notice is displayed on the plugins.php and the PayPal Payments settings pages to warn the user about the potential impact of the available update.

@burdulixda burdulixda requested a review from Dinamiko July 10, 2024 06:28
@burdulixda burdulixda marked this pull request as ready for review July 10, 2024 13:12
@Dinamiko Dinamiko merged commit 90eeecc into trunk Jul 23, 2024
8 checks passed
@Dinamiko Dinamiko deleted the PCP-3191-introduce-notice-for-paypal-payments-3-0-update-warning branch July 23, 2024 12:29
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.

None yet

2 participants