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

added check to validate password #696

Closed
wants to merge 1 commit into from

Conversation

somyaup
Copy link

@somyaup somyaup commented May 27, 2018

Description

Added check for password validation(Passwords must use at least three of the four available character types: lowercase letters, uppercase letters, numbers, and symbols.)

Fixes #644

Type of Change:

Delete irrelevant options.

  • Code

Code/Quality Assurance Only

How Has This Been Tested?

Describe the tests you ran to verify your changes. Provide instructions or GIFs so we can reproduce. List any relevant details for your test.

Checklist:

Delete irrelevant options.

  • My PR follows the style guidelines of this project
  • I have performed a self-review of my own code or materials
  • I have commented my code or provided relevant documentation, particularly in hard-to-understand areas

Code/Quality Assurance Only

  • My changes generate no new warnings
  • New and existing unit tests pass locally with my changes

@coveralls
Copy link

coveralls commented May 27, 2018

Coverage Status

Coverage decreased (-0.4%) to 42.021% when pulling dbca0d4 on somyaup:validate_password into 39285b6 on systers:develop.

@somyaup somyaup closed this May 27, 2018
@anitab-org anitab-org deleted a comment May 27, 2018
@anitab-org anitab-org deleted a comment May 27, 2018
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.

None yet

3 participants