Skip to content

35 register validation#62

Merged
khoadnguyen merged 6 commits intodevfrom
35-register-validation
Oct 2, 2018
Merged

35 register validation#62
khoadnguyen merged 6 commits intodevfrom
35-register-validation

Conversation

@jthk1106
Copy link
Copy Markdown
Contributor

@jthk1106 jthk1106 commented Oct 1, 2018

issue 35
Completed form validation for all input fields except password confirmation. Still working on that one. Invalid highlights around input, invalid messages for inputs, and disabled register button for incomplete form are all styled.

issue 44
Fixed "INTRANSITION" text in .card-title header from exceeding width in ipad pro portrait mode. Need to discuss with Alex if resolution is acceptable.

@jthk1106 jthk1106 added the code review This PR is ready for code review label Oct 1, 2018
@jthk1106 jthk1106 added this to the Week 3 milestone Oct 1, 2018
@jthk1106 jthk1106 self-assigned this Oct 1, 2018
@jthk1106 jthk1106 requested a review from khoadnguyen October 1, 2018 09:00
Copy link
Copy Markdown
Contributor

@khoadnguyen khoadnguyen left a comment

Choose a reason for hiding this comment

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

Code looks good, ⚡️

@khoadnguyen khoadnguyen merged commit a068103 into dev Oct 2, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

code review This PR is ready for code review

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants