Skip to content
This repository was archived by the owner on Jul 6, 2020. It is now read-only.

update signup component #236

Closed
wants to merge 1 commit into from
Closed

Conversation

amanex007
Copy link
Contributor

in response to #235

Fixes #

Changes proposed in this pull request:

@codecov-io
Copy link

Codecov Report

Merging #236 into master will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##           master     #236   +/-   ##
=======================================
  Coverage   51.73%   51.73%           
=======================================
  Files          66       66           
  Lines        3659     3659           
  Branches      413      413           
=======================================
  Hits         1893     1893           
  Misses       1671     1671           
  Partials       95       95

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update b03bdc4...90e6d7d. Read the comment docs.

@@ -51,7 +51,7 @@
<i *ngIf="authService.canShowPassword" class="fa fa-eye-slash pointer"></i>
</span>
<label for="password" [class.active]="ispasswordFocused">
<strong>Password-minlength:8 required</strong>
<strong>Password</strong>
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

How about changing this to

Password (minimum 8 Characters)

@pushkalkatara
Copy link

Hi @amanex007 Can you make a single PR for both. (ie single pr for #235 and #236)

@amanex007 amanex007 closed this Dec 16, 2019
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.

4 participants