-
Notifications
You must be signed in to change notification settings - Fork 67
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
Polish logic of redirect when click Set Up WooPayments from WC Home #7659
Polish logic of redirect when click Set Up WooPayments from WC Home #7659
Conversation
Test the buildOption 1. Jetpack Beta
Option 2. Jurassic Ninja - available for logged-in A12s🚀 Launch a JN site with this branch 🚀 ℹ️ Install this Tampermonkey script to get more options. Build info:
Note: the build is updated when a new commit is pushed to this PR. |
Size Change: 0 B Total Size: 1.44 MB ℹ️ View Unchanged
|
…ore-polish-current-logic
…ore-polish-current-logic
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.
Nice to see this proactivity and push for perfection, @oaratovskyi! 💪
All the testing instructions check out! And the code changes make for more resilient system! 🚢 it!
Related #7625
Follow up of #7652 with intent to improve the solution for more account statuses by making decision based on
details_submitted
value, see p1699436266692369/1699365759.939169-slack-C03KTTK2YMADescription
Merchants currently observe an issue with Set up WooPayments task: when account completed the KYC / are rejected / pending - have Stripe account connected, the link redirect them to
payments/onboarding
which results in Not allowed page errorChanges proposed in this Pull Request
I decided that the best experience for merchants will be to get redirected to
payments/overview
page if the account connected to Stripe in any status. If we try to redirect the merchant topayments/onboarding
page when they have Stripe account connected, they'll face Not allowed page. That's why we should redirect even partially onboarded users (details_submitted = false) to the overview page. They'll be able to proceed from there by hitting Finish Setup buttonImportant
Tests are not added since there is no tests for
maybe_handle_onboarding
function. Creating tests for it is out of scope for this PR. It'll be done in #7654Testing instructions
Pre-requisites
tweak/7625-set-up-woopayments-link-from-core-polish-current-logic
Stripe account not connected
Complete
statuspayments/overview
pageStripe account connected (pending)
Pending
statuspayments/overview
pageStripe KYC left early / partially onboarded accounts
payments/overview
pagenpm run changelog
to add a changelog file, choosepatch
to leave it empty if the change is not significant. You can add multiple changelog files in one PR by running this command a few times.Post merge