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

Cherry pick 51541 into release/9.4 #51615

Merged
merged 2 commits into from
Sep 23, 2024
Merged

Conversation

github-actions[bot]
Copy link
Contributor

This PR cherry-picks the following PRs into the release branch:

vladolaru and others added 2 commits September 23, 2024 11:37
* Minor cleanup

* refact: Rename must_be_visible to is_incentive_visible for clarity

* refact: Rename register_payments_welcome_page to register_menu_and_page for clarity

* Don't show the incentive page if the user can't manage WC

* Introduce menu item when no incentive

* Don't recommend additional gateways when WCPay is present

* Render gateway suggestions on the WCPay task page, also

* Add changelog entry

* Remove short array syntax

* Lint fixes

* Simplify Payments task view logic

* Simplify WooPayments task view logic

* Improve WooPayments task complete logic

* WooPayments can't be connected or onboarded if it is not active

* refact: Rename private static method

* Improve logic to determine the Payments task slug by factoring in the extended task list also

* docs: Add details about our use of the Payments task logic

* Add type hint

* Do not recommend offline methods when WooPayments is supported

* On suggestion enablement redirect to the same task page

* Lint fixes

* test: Fix unit tests

* Maintain deprecated navigation logic
@github-actions github-actions bot added the metric: code freeze exception A tracking label for PRs that break the release code freeze. label Sep 23, 2024
@naman03malhotra naman03malhotra merged commit de337d5 into release/9.4 Sep 23, 2024
@naman03malhotra naman03malhotra deleted the cherry-pick-9.4.0/51541 branch September 23, 2024 15:02
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
metric: code freeze exception A tracking label for PRs that break the release code freeze.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants