Skip to content
This repository has been archived by the owner on Dec 14, 2021. It is now read-only.

Fixed Issues #1152, #1156, #1233, and #1172: Fixed issues related to Discard, Save, and Reveal password in Edit mode #1234

Closed
wants to merge 11 commits into from

Conversation

KrisRJack
Copy link
Contributor

@KrisRJack KrisRJack commented May 23, 2020

Fixes #1152 #1156 #1172 #1233

This PR fixes all of the following issue related to Discard, Save, and Reveal password in Edit mode. Without this PR, there are 4 major issues that I noticed (some were already reported):

  1. Discard does not revert text back to the original string in the username and password fields. Before this PR, the user had to physically click out of the page and open it again to see the discarded changes.
  2. When Save is clicked by the user, the Hide/Reveal password function does not update the password value to the updated string. The user has to exit and re-enter the page to see the changes to the password.
  3. The password field saves the "•" values when a password is hidden and 'Save' has been clicked.
  4. There is no error handling for empty fields when saving a username or password.

This PR fixes all the above issues that I noticed related to Edit mode. Below are screen shots of the error handling (with error handling, I also included haptic feedback).

Screenshot

Simulator Screen Shot - iPhone 11 - 2020-05-23 at 22 29 45

To Do

  • add “WIP” to the PR title if pushing up but not complete nor ready for review
  • double check the original issue to confirm it is fully satisfied
  • add testing notes and screenshots in PR description to help guide reviewers
  • add unit tests
    • optional: consider adding integration tests (UI specs)
  • consider running this branch in the simulator and check for warnings
  • request the "UX" team perform a design review (if/when applicable)
  • make sure CI builds are passing (e.g.: fix lint and other errors)

@KrisRJack KrisRJack requested a review from a team as a code owner May 23, 2020 15:11
@KrisRJack KrisRJack changed the title Fixes Issues #1156, #1233, and #1172 Fixed Issues #1156, #1233, and #1172: Fixed issues related to Discard, Save, and Reveal password in Edit mode May 23, 2020
@KrisRJack KrisRJack changed the title Fixed Issues #1156, #1233, and #1172: Fixed issues related to Discard, Save, and Reveal password in Edit mode Fixed Issues #1152, #1156, #1233, and #1172: Fixed issues related to Discard, Save, and Reveal password in Edit mode May 24, 2020
@kaylagalway
Copy link
Contributor

Hey @Krisjackson if you have time - could you resolve the merge conflicts on this PR?

@KrisRJack
Copy link
Contributor Author

@kaylagalway Should be fixed now. Let me know if it works

@kaylagalway
Copy link
Contributor

I'll be offline for two weeks on short term leave, but will be reviewing whatever pull requests I still haven't merged when I get back. A good amount of them should be in the late June release we will be doing. Thanks for all your hard work!

@KrisRJack
Copy link
Contributor Author

No worries at all. Thanks for reviewing my code and allowing me to contribute!

@st3fan st3fan closed this Jan 27, 2021
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Edit Credential: Error Messaging
3 participants