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

fix: remove OneSignal v2 from modal checkout #2061

Open
wants to merge 1 commit into
base: trunk
Choose a base branch
from

Conversation

laurelfulford
Copy link
Contributor

@laurelfulford laurelfulford commented Feb 24, 2025

All Submissions:

Changes proposed in this Pull Request:

This PR expands on the change made here to remove OneSignal from the modal checkout, to make sure both v2 and v3 of the plugin code is removed.

See 1200550061930446-as-1209206364302813

How to test the changes in this Pull Request:

  1. To recreate the testing setup for this bug, start with a site that hasn't run OneSignal before. You can also ping me for a JN site with this setup.
  2. Download and install v 2.4.4 of OneSignal from here.
  3. Sign up for a free account.
  4. Connected the account to your site (you have to fill out some forms and the site's URL, then copy the OneSignal App ID and OneSignal REST API Key to the WP Admin > One Signal Push page on your website).
  5. Enable the Subscription bell from the WP Admin > One Signal Push page:

CleanShot 2025-02-24 at 14 05 46

  1. Save the settings
  2. View the site on the front-end and confirm that the bell is appearing.
  3. Upgrade the plugin from 2.4.4 to the latest version. I ran into issues with this for some reason, but it worked if I downloaded some of the versions in between from here and uploaded them (though 3.0.0 threw a fatal).
  4. Run through a modal checkout flow and confirm the bell appears in the modal.
  5. Apply this PR.
  6. Repeat step 8 and confirm that the bell no longer appears in the modal checkout.
  7. Test on a second site with the latest version of OneSignal from the get-go, so it's loading the v3 version of the code.
  8. Smoke test: retest reCAPTCHA for WooCommerce by following the steps here to make sure it's also still stripped out of the modal checkout, even though the hook changed. You can find a copy of that plugin attached to the Asana task.

Other information:

  • Have you added an explanation of what your changes do and why you'd like us to include them?
  • Have you written new tests for your changes, as applicable?
  • Have you successfully ran tests with your changes locally?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant