-
Notifications
You must be signed in to change notification settings - Fork 1.3k
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
Plans on JP: Checkout Success #18497
Conversation
|
App Name | ![]() |
|
Flavor | Jalapeno | |
Build Type | Debug | |
Version | pr18497-92cd6cf | |
Commit | 92cd6cf | |
Direct Download | jetpack-prototype-build-pr18497-92cd6cf.apk |
|
App Name | ![]() |
|
Flavor | Jalapeno | |
Build Type | Debug | |
Version | pr18497-92cd6cf | |
Commit | 92cd6cf | |
Direct Download | wordpress-prototype-build-pr18497-92cd6cf.apk |
Thanks, @ravishanker, the flow overall works well! 👍 Couple observations:
Android flowtest.android.moviOS flow (success not yet implemented)Simulator.Screen.Recording.-.iPhone.14.-.2023-05-24.at.16.28.24.mp4 |
👋 @staskus - Thanks for testing
It should or it shouldn't return to plans? Point 3 on that issue is saying it should return to plans view on back press.
🤔 Looks like site is not refreshing |
@ravishanker, I think I misspoke. On Android, we return from Checkout to Domains view after tapping back. Ideally, it would return to the Plans view.
I will test again today, it's hard to tell, maybe it was related to the sandbox store and it's not actually an issue. |
I do get a receipt in email when testing upgrades, that upgrade happened. It's not clear, if the actual upgrade happens when using sandbox! I checked with test accounts on Test Plan pc8eDl-WC-p2, Also, checked using Store Admin, the upgrade doesn't seem to actually happen, it goes through test tables apparently, not actual ones PCYsg-MaN-p2, and PCYsg-IA-p2 |
@ravishanker thanks for the information. I tested again and got the same results for the same site. The card disappears after completing the purchase on iOS and doesn't disappear on Android. On Android, I also get "Verify email" popup after completing the purchase, although I checked the email and I only got the receipt but not a verification email. Not sure if it's a related issue. Are you getting the same result on Android yourself? ios.test.again.mp4android.test.again.mov |
Generated by 🚫 dangerJS |
Add a new activity result request code to avoid conflict with existing domain purchase
It doesn't disappear too if not pointing to the sandbox. It has to do with store Sandbox vs Live discrepancy! The transaction is created on sandbox, while dashboard site is live source. From the FG: Testing the Store PCYsg-IA-p2
I've checked on the store admin, the subscription is being added successfully on store sandbox. The email is not related to plans purchase, it's just a conflict with the plugin installation feature, fixed it now. By any chance is your iOS simulator/device pointing to sandbox, while android emulator/device doesn't? How to connect android emulator/device to sandbox paqN3M-5m-p2 |
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.
@ravishanker, thanks for the answers! It does point to the sandbox, otherwise, the payment wouldn't go through with the testing cards. After I make a purchase on Android, it does appear on SA but the API still tells that the user is on a free plan. It's strange to me that that doesn't happen on iOS. Not sure what could be the difference, since we both finish the purchase on the same web view 🤔 |
I tested further and it seems to be related to the sandbox store configuration rather than anything else. On iOS, I observe the same behavior if I kill and relaunch the app after the purchase. |
Updates checkout success screen to new design, and removes the old one
Fixes: #18422
To test:
Feature Flag
Store sandbox
Success screen
Regression Notes
Potential unintended areas of impact
None
What I did to test those areas of impact (or what existing automated tests I relied on)
Unit tests and manual tests
What automated tests I added (or what prevented me from doing so)
Updated and added unit tests
PR submission checklist:
RELEASE-NOTES.txt
if necessary.UI Changes testing checklist: