Skip to content

show specific err msg on password form#4945

Merged
puneetlath merged 2 commits into
Expensify:mainfrom
thesahindia:thesahindia/ui-ux/passwordForm
Sep 2, 2021
Merged

show specific err msg on password form#4945
puneetlath merged 2 commits into
Expensify:mainfrom
thesahindia:thesahindia/ui-ux/passwordForm

Conversation

@thesahindia

Copy link
Copy Markdown
Member

Details

Proposal: #4856 (comment)
Approval : #4856 (comment)
As discussed, I've added conditions for 3 specific error messages on password form.

Fixed Issues

$ #4856

QA Steps

  1. Launch the app
  2. Enter email or phone number & proceed
  3. Click on signin without entering password

Tested On

  • Web
  • Mobile Web
  • Desktop
  • iOS
  • Android

Screenshots

Web

web

Mobile Web

Mobile web

Desktop

iOS

Android

android

@thesahindia thesahindia requested a review from a team as a code owner August 31, 2021 11:44
@MelvinBot MelvinBot requested review from puneetlath and removed request for a team August 31, 2021 11:44

@puneetlath puneetlath 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.

A couple minor comments. Up to you whether you agree or not.

Also, just waiting for a response on this: #4856 (comment)

Comment thread src/pages/signin/PasswordForm.js
Comment thread src/pages/signin/PasswordForm.js Outdated
@thesahindia

Copy link
Copy Markdown
Member Author

Making the required changes!

@puneetlath puneetlath 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.

Looks great!

@puneetlath puneetlath merged commit 175a1f2 into Expensify:main Sep 2, 2021
@OSBotify

OSBotify commented Sep 2, 2021

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

OSBotify commented Sep 3, 2021

Copy link
Copy Markdown
Contributor

🚀 Deployed to staging by @puneetlath in version: 1.0.92-1 🚀

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

@botify

botify commented Sep 3, 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-09-10. 🎊

@OSBotify

OSBotify commented Sep 4, 2021

Copy link
Copy Markdown
Contributor

🚀 Deployed to production by @roryabraham in version: 1.0.93-1 🚀

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.

4 participants