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

[CP Stg] Add Venmo only if sending money #5661

Merged
merged 1 commit into from
Oct 5, 2021

Conversation

nickmurray47
Copy link
Contributor

@nickmurray47 nickmurray47 commented Oct 5, 2021

Details

This PR needs to be tested on a staging environment since simulators cannot download Venmo locally from the App Store.

Fixed Issues

$ #5655

Tests / QA Steps

Precondition: installed the Venmo app and log in with it.
2. Log in with phone account number, e.g. (+14016597742)
3. Click on user with phone number e.g. (used +15082211196) and click Request Money
4. Put amount and Continue
5. Verify only one green button appears with Request $X

Tested On

  • Web
  • Mobile Web
  • Desktop
  • iOS
  • Android

Screenshots

Web

Mobile Web

Desktop

iOS

Android

@nickmurray47 nickmurray47 self-assigned this Oct 5, 2021
@nickmurray47 nickmurray47 requested a review from a team as a code owner October 5, 2021 18:27
@MelvinBot MelvinBot requested review from thienlnam and removed request for a team October 5, 2021 18:27
@github-actions
Copy link
Contributor

github-actions bot commented Oct 5, 2021

⚠️ ⚠️ Heads up! This pull request has the CP Staging label. ⚠️ ⚠️
Merging it will cause it to be immediately deployed to staging, even if the open StagingDeployCash deploy checklist is locked.

@nickmurray47 nickmurray47 requested a review from Jag96 October 5, 2021 19:00
@thienlnam thienlnam merged commit e99bd02 into main Oct 5, 2021
@thienlnam thienlnam deleted the nmurray-put-venmo-behind-send branch October 5, 2021 19:25
github-actions bot pushed a commit that referenced this pull request Oct 5, 2021
@OSBotify
Copy link
Contributor

OSBotify commented Oct 5, 2021

🚀 Cherry-picked to staging by @thienlnam in version: 1.1.5-1 🚀

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

@Expensify/applauseleads please QA this PR and check it off on the deploy checklist if it passes.

@isagoico
Copy link

isagoico commented Oct 5, 2021

Retest was a pass 🎉 Checking it off the list

@OSBotify
Copy link
Contributor

OSBotify commented Oct 6, 2021

🚀 Deployed to production by @chiragsalian in version: 1.1.5-1 🚀

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

@OSBotify
Copy link
Contributor

OSBotify commented Oct 6, 2021

🚀 Deployed to staging by @thienlnam in version: 1.1.5-2 🚀

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

@OSBotify
Copy link
Contributor

OSBotify commented Oct 7, 2021

🚀 Deployed to production by @chiragsalian in version: 1.1.6-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