[Payment card / Subscription] Implement “Your plan” section (UI)#42690
[Payment card / Subscription] Implement “Your plan” section (UI)#42690amyevans merged 16 commits intoExpensify:mainfrom
Conversation
|
CC: @fabioh8010 |
src/pages/settings/Subscription/SaveWithExpensifyButton/index.tsx
Outdated
Show resolved
Hide resolved
amyevans
left a comment
There was a problem hiding this comment.
1 small update to go from my perspective
amyevans
left a comment
There was a problem hiding this comment.
Sorry, one other quick thing I just remembered!
|
Everything looks good here so far. I'll retest after the final set of changes and then complete the PR checklist. cc @amyevans |
|
@amyevans PR updated again! |
|
@JKobrynski NAB. Just confirming that if the user does not have a Collect or Control workspace, navigating to the subscription page would result in the "Hmm... not here" screen being displayed. |
akinwale
left a comment
There was a problem hiding this comment.
LGTM.
Can be merged after the conflicts are resolved.
|
@akinwale conflicts resolved! |
|
✋ This PR was not deployed to staging yet because QA is ongoing. It will be automatically deployed to staging after the next production release. |
|
🚀 Deployed to production by https://github.com/Julesssss in version: 1.4.79-11 🚀
|
Details
To access this newly created component, paste the following link into the browser
https://dev.new.expensify.com:8082/settings/subscription
or add this effect to
InitialSettingsPage.tsxFixed Issues
$ #38620
PROPOSAL: N/A
Tests
Prerequisites
To test this component you are going to need a Collect workspace and then a Control workspace (ideally in that order because if you have both at the same time, Control will be displayed over Collect). To create either one go to https://staging.expensify.com and then to Settings -> Workspaces and click the New Workspace button. In the next step you can create a Collect/Control workspace, that will be available in new dot (on the same account) after login.
Offline tests
N/A
QA Steps
Same as Tests section above
PR Author Checklist
### Fixed Issuessection aboveTestssectionOffline stepssectionQA stepssectiontoggleReportand notonIconClick)myBool && <MyComponent />.src/languages/*files and using the translation methodSTYLE.md) were followedAvatar, I verified the components usingAvatarare working as expected)StyleUtils.getBackgroundAndBorderStyle(theme.componentBG))Avataris modified, I verified thatAvataris working as expected in all cases)Designlabel and/or tagged@Expensify/designso the design team can review the changes.ScrollViewcomponent to make it scrollable when more elements are added to the page.mainbranch was merged into this PR after a review, I tested again and verified the outcome was still expected according to theTeststeps.Screenshots/Videos
Android: Native
Collect
Control
Android: mWeb Chrome
Collect
Control
iOS: Native
Collect
Control
iOS: mWeb Safari
Collect
Control
MacOS: Chrome / Safari
Collect
Control
MacOS: Desktop
Collect
Control