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

Update Get started button to Finish setup when a user has draft data saved in the VBA flow #5258

Merged
merged 9 commits into from
Sep 28, 2021

Conversation

luacmartins
Copy link
Contributor

@luacmartins luacmartins commented Sep 14, 2021

Details

The button in the Workspace card page should change to Finish setup when a user has entered data into the form and closes the modal.

Fixed Issues

$ #4892

Tests

  1. Login to an account with a Workspace and without an Expensify Card enabled or bank account setup.
  2. Navigate to Settings > Select Workspace > Get Started.
  3. Start filling out the form.
  4. Close the form and notice that the button changed to Finish setup.

QA Steps

Steps above.

Tested On

  • Web
  • Mobile Web
  • Desktop
  • iOS
  • Android

Screenshots

Web

web.mov

Mobile Web

Desktop

desktop.mov

iOS

Android

@luacmartins luacmartins self-assigned this Sep 14, 2021
@luacmartins luacmartins changed the title Update Get started button to Finish setup as soon as a user's information have been saved for a step in VBA flow Update Get started button to Finish setup when a user has draft data saved in the VBA flow Sep 22, 2021
@luacmartins luacmartins requested a review from a team September 22, 2021 21:03
@luacmartins luacmartins marked this pull request as ready for review September 22, 2021 21:03
@MelvinBot MelvinBot requested review from AndrewGable and removed request for a team September 22, 2021 21:04
@luacmartins luacmartins requested a review from a team as a code owner September 22, 2021 21:13
@MelvinBot MelvinBot requested review from Beamanator and removed request for a team September 22, 2021 21:13
@luacmartins luacmartins removed the request for review from Beamanator September 22, 2021 21:16
@AndrewGable AndrewGable changed the title Update Get started button to Finish setup when a user has draft data saved in the VBA flow [HOLD]Update Get started button to Finish setup when a user has draft data saved in the VBA flow Sep 22, 2021
@AndrewGable
Copy link
Contributor

Placing on hold, can you ping me when it's ready for merge and review? Thanks!

@luacmartins luacmartins changed the title [HOLD]Update Get started button to Finish setup when a user has draft data saved in the VBA flow Update Get started button to Finish setup when a user has draft data saved in the VBA flow Sep 27, 2021
@luacmartins
Copy link
Contributor Author

No longer on hold! @AndrewGable all yours for review!

@AndrewGable
Copy link
Contributor

Looking real good! I couldn't find anything to comment on 😄

@AndrewGable AndrewGable merged commit 2036c3f into main Sep 28, 2021
@AndrewGable AndrewGable deleted the cmartins-finishSetup branch September 28, 2021 21:00
@OSBotify
Copy link
Contributor

✋ This PR was not deployed to staging yet because QA is ongoing. It will be automatically deployed to staging after the next production release.

Copy link
Contributor

@Jag96 Jag96 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

One question and 1 nab, lgtm

src/libs/actions/BankAccounts.js Show resolved Hide resolved
src/pages/workspace/WorkspaceCardPage.js Show resolved Hide resolved
@OSBotify
Copy link
Contributor

🚀 Deployed to staging by @AndrewGable in version: 1.1.3-2 🚀

platform result
🤖 android 🤖 success ✅
🖥 desktop 🖥 success ✅
🍎 iOS 🍎 success ✅
🕸 web 🕸 success ✅

@isagoico
Copy link

@luacmartins Hello! I think we're blocked from testing this. When creating an account with gmail it asks us to add a new domain with a work email. I think this might need to be internal QA.

@luacmartins
Copy link
Contributor Author

@isagoico I updated the testing instructions. Could you please try again?

@OSBotify
Copy link
Contributor

OSBotify commented Oct 4, 2021

🚀 Deployed to production by @chiragsalian in version: 1.1.4-0 🚀

platform result
🤖 android 🤖 success ✅
🖥 desktop 🖥 success ✅
🍎 iOS 🍎 success ✅
🕸 web 🕸 success ✅

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants