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

password match error fixed #4164

Merged
merged 2 commits into from
Jul 22, 2021
Merged

password match error fixed #4164

merged 2 commits into from
Jul 22, 2021

Conversation

ABee-Tech
Copy link
Contributor

Details

Check out the comments in this issue for more detail: #3976

Fixed Issues

$ #3976

Tests

  1. Enter email on the first screen after you have downloaded the app.
  2. The magic link is sent to your email.
  3. Tap on the magic link to open it in Expensify.cash app.
  4. Tap on confirm password field.
  5. Type anything and tap on enter your new password field.
  6. Type password as directed as at least 8 characters, 1 capital letter, 1 lowercase letter, 1 number.
  7. Tap on confirm the password field and clear all the text on the "Confirm Password" field then the screen will not become blank again.

QA Steps

Same as tests

Tested On

  • Web
  • Mobile Web
  • Desktop
  • iOS
  • Android

Screenshots

Android

After.1.mp4

@ABee-Tech ABee-Tech requested a review from a team as a code owner July 21, 2021 16:40
@MelvinBot MelvinBot requested review from iwiznia and removed request for a team July 21, 2021 16:40
@github-actions
Copy link
Contributor

github-actions bot commented Jul 21, 2021

CLA Assistant Lite bot All contributors have signed the CLA ✍️ ✅

@ABee-Tech
Copy link
Contributor Author

I have read the CLA Document and I hereby sign the CLA

@ABee-Tech
Copy link
Contributor Author

@iwiznia I have done changes. Please review the changes.

iwiznia
iwiznia previously approved these changes Jul 22, 2021
@iwiznia
Copy link
Contributor

iwiznia commented Jul 22, 2021

Seems you have linting errors, can you fix those? Also keep an eye on the tests that are still running.

@ABee-Tech
Copy link
Contributor Author

I didn't get what exactly the lint is telling about the trailing comma. Can you clarify so that I can push it?

@ABee-Tech
Copy link
Contributor Author

Okay, I got it I will fix the lint. It is searching comma like this:

    showPasswordMatchError() {
        return Boolean(
            !this.doPasswordsMatch()
            && this.state.shouldShowPasswordConfirmError
            && this.state.confirmNewPassword,
        );
    }

@iwiznia
Copy link
Contributor

iwiznia commented Jul 22, 2021

Yeah, but it should not be needed... I think the linter is making a mistake.

@iwiznia
Copy link
Contributor

iwiznia commented Jul 22, 2021

Anyway, go ahead and add it.

@ABee-Tech
Copy link
Contributor Author

Ohh, I added it. Is that ok?

@iwiznia
Copy link
Contributor

iwiznia commented Jul 22, 2021

Yeah yeah, we can figure out what's wrong with the linter later (if we care)

@ABee-Tech
Copy link
Contributor Author

Okay, thank you.

@iwiznia iwiznia merged commit 4facad0 into Expensify:main Jul 22, 2021
@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.

@ABee-Tech ABee-Tech deleted the password-match-error-fix branch July 22, 2021 15:02
@OSBotify
Copy link
Contributor

🚀 Deployed to staging in version: 1.0.79-5🚀

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

@OSBotify
Copy link
Contributor

🚀 Deployed to production in version: 1.0.80-2🚀

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.

3 participants