Double confirmation of account password #570

Open
opened 2019-06-01 20:53:16 +02:00 by tzarebczan · 2 comments
tzarebczan commented 2019-06-01 20:53:16 +02:00 (Migrated from github.com)

The Issue

Probably a good idea to ask for double confirmation like we do in the desktop app. This may add more friction, but it's also better than typing it incorrectly and losing the credits.

@kauffj edit attack: also make entering your password opt-in, rather than default

System Configuration

  • Device:
  • Android version:

Anything Else

Screenshots

Internal Use

Acceptance Criteria

Definition of Done

  • Tested against acceptance criteria
  • Tested against the assumptions of the user story
  • The project builds without errors
  • Unit tests are written and passing
  • Tests on devices/browsers listed in the issue have passed
  • QA performed & issues resolved
  • Refactoring completed
  • Any configuration or build changes documented
  • Documentation updated
  • Peer Code Review performed
<!-- Thanks for reporting an issue to LBRY and helping us improve! To make it possible for us to help you, please fill out below information carefully. Before reporting any issues, please make sure that you're using the latest version. We are also available on live chat at https://chat.lbry.com --> ## The Issue Probably a good idea to ask for double confirmation like we do in the desktop app. This may add more friction, but it's also better than typing it incorrectly and losing the credits. @kauffj edit attack: also make entering your password opt-in, rather than default ## System Configuration - Device: - Android version: ## Anything Else <!-- Include anything else that does not fit into the above sections --> ## Screenshots <!-- If a screenshot would help explain the bug, please include one or two here --> ## Internal Use ### Acceptance Criteria 1. 2. 3. ### Definition of Done - [ ] Tested against acceptance criteria - [ ] Tested against the assumptions of the user story - [ ] The project builds without errors - [ ] Unit tests are written and passing - [ ] Tests on devices/browsers listed in the issue have passed - [ ] QA performed & issues resolved - [ ] Refactoring completed - [ ] Any configuration or build changes documented - [ ] Documentation updated - [ ] Peer Code Review performed
akinwale commented 2019-06-03 14:05:04 +02:00 (Migrated from github.com)

@tzarebczan What do you think about making the password field visible by default instead and then the user can choose to hide it afterwards? Would like to keep the first run / enroll process as simple as possible by just using one field per screen.

@tzarebczan What do you think about making the password field visible by default instead and then the user can choose to hide it afterwards? Would like to keep the first run / enroll process as simple as possible by just using one field per screen.
tzarebczan commented 2019-06-09 00:00:44 +02:00 (Migrated from github.com)

If we are allowing blank passwords, I think we should go with double. It's even normal on many other apps and sites, let alone crypto ones. Already had one person forget their password..think this will make them a bit more aware when typing. If we had 2fa for recovery, I'd say 1 is enough , but otherwise there's no way for us to get back the pw.

If we are allowing blank passwords, I think we should go with double. It's even normal on many other apps and sites, let alone crypto ones. Already had one person forget their password..think this will make them a bit more aware when typing. If we had 2fa for recovery, I'd say 1 is enough , but otherwise there's no way for us to get back the pw.
Sign in to join this conversation.
No milestone
No project
No assignees
1 participant
Notifications
Due date
The due date is invalid or out of range. Please use the format "yyyy-mm-dd".

No due date set.

Dependencies

No dependencies set.

Reference: LBRYCommunity/lbry-android#570
No description provided.