Skip to content

Add confirmation step for set_password prompt #321

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.

Sign up for GitHub

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

Jump to bottom
Merged
merged 3 commits into from
Jun 12, 2017
Merged

Add confirmation step for set_password prompt #321

merged 3 commits into from
Jun 12, 2017

Conversation

@eywalker
Copy link
Contributor

@eywalker eywalker commented Jun 12, 2017

Fix #320

  • When using dj.set_password with prompts, the prompt message is now more helpful and also includes confirmation step to decrease the chance of changing to a password containing typos
  • Increment to version 0.7.1

@coveralls
Copy link

coveralls commented Jun 12, 2017 •
edited
Loading

Coverage Status

Coverage remained the same at 93.103% when pulling be6c03d on eywalker:master into 332e061 on datajoint:master.

@dimitri-yatsenko dimitri-yatsenko merged commit 923c2a7 into datajoint:master Jun 12, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@dimitri-yatsenko dimitri-yatsenko dimitri-yatsenko approved these changes

@fabiansinz fabiansinz Awaiting requested review from fabiansinz

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

3 participants

@eywalker @coveralls @dimitri-yatsenko