Skip to content

Show a growl when trying to create a chat with an invalid phone number#4525

Merged
Beamanator merged 1 commit into
Expensify:mainfrom
rdjuric:growlOnInvalidNumber
Aug 11, 2021
Merged

Show a growl when trying to create a chat with an invalid phone number#4525
Beamanator merged 1 commit into
Expensify:mainfrom
rdjuric:growlOnInvalidNumber

Conversation

@rdjuric

@rdjuric rdjuric commented Aug 10, 2021

Copy link
Copy Markdown
Contributor

Details

Shows an error Growl when trying to create a invalid chat.

Fixed Issues

$ #2934

Tests

  1. Search for an invalid number (like +5512)
  2. Click on the result
  3. A Growl shows up with the correct error message

QA Steps

Tested On

  • Web
  • Mobile Web
  • Desktop
  • iOS
  • Android

Screenshots

Web

web.mov

Mobile Web

mWeb.mp4

Desktop

desktop.mov

iOS

iOS.mp4

Android

Android.mp4

@rdjuric rdjuric marked this pull request as ready for review August 10, 2021 20:38
@rdjuric rdjuric requested a review from a team as a code owner August 10, 2021 20:38
@MelvinBot MelvinBot requested review from cead22 and removed request for a team August 10, 2021 20:38
@rdjuric rdjuric changed the title growl when creating invalid chat Show a growl when trying to create a chat with an invalid phone number Aug 10, 2021
@Beamanator Beamanator self-requested a review August 11, 2021 07:46

@Beamanator Beamanator left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

LGTM 👍 (sorry @cead22 , stealing this)

@Beamanator Beamanator merged commit 83dd58e into Expensify:main Aug 11, 2021
@OSBotify

Copy link
Copy Markdown
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.

@OSBotify

Copy link
Copy Markdown
Contributor

🚀 Deployed to staging by @Beamanator in version: 1.0.83-2 🚀

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

@botify

botify commented Aug 17, 2021

Copy link
Copy Markdown

This has been deployed to production and is now subject to a 7-day regression period.
If no regressions arise, payment will be issued on 2021-08-24. 🎊

@OSBotify

Copy link
Copy Markdown
Contributor

🚀 Deployed to production by @isagoico in version: 1.0.85-9 🚀

platform result
🤖 android 🤖 failure ❌
🖥 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.

4 participants