-
Notifications
You must be signed in to change notification settings - Fork 10.8k
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 wrong payment tasks display logic #34892
Conversation
Test Results SummaryCommit SHA: 6079093
To view the full E2E test report, click here. To view all test reports, visit the WooCommerce Test Reports Dashboard. |
…nt task is completed
9813c8d
to
6079093
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Fix seems to work correctly! 🚢
Discovered another issue p1664779377044819/1664765407.476729-slack-C01SFMVEYAK but will create another issue for that.
Thanks @chihsuan, It tested well for me too! I also checked it using a country not eligible for WC Pay and in that case the |
Hi @chihsuan, thanks for merging this pull request. Please take a look at these follow-up tasks you may need to perform:
|
* Fix the wc pay view logic so that we don't display it even when payment task is completed * Add changelog
* Fix wrong payment tasks display logic (#34892) * Fix the wc pay view logic so that we don't display it even when payment task is completed * Add changelog * Prep for cherry pick 34892 Co-authored-by: Chi-Hsuan Huang <chihsuan.tw@gmail.com> Co-authored-by: WooCommerce Bot <no-reply@woocommerce.com>
All Submissions:
Changes proposed in this Pull Request:
Closes #34893 .
This PR fixes the wrong payment task display logic.
Set up payments
should not be displayed when WC pay task is completed and vice versa.Test zip: https://github.com/woocommerce/woocommerce/actions/runs/3156126725
How to test the changes in this Pull Request:
WooCommerce > Home
Set up WooCommerce Payments
taskSet up WooCommerce Payments
task is still displayed and marked as completed whileSet up payments
task is not shown.Other information:
pnpm --filter=<project> run changelog add
?FOR PR REVIEWER ONLY: