Skip to content
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

Password message #429

Merged
merged 2 commits into from
Oct 25, 2014
Merged

Password message #429

merged 2 commits into from
Oct 25, 2014

Conversation

rgrigga
Copy link
Contributor

@rgrigga rgrigga commented Oct 3, 2014

if passwords don't match, return "password confirm" message rather than "wrong email confirmation code"

rgriss added 2 commits October 2, 2014 22:08
@rgriss
Update UserValidator.php
1af870a 
update to password_confirmation
@rgriss
Update confide.php
ace240b 
password confirmation
@coveralls
Copy link

Coverage Status

Coverage decreased (-0.24%) when pulling ace240b on rgrigga:patch-2 into c1280c4 on Zizaco:master.

@rgrigga
Copy link
Contributor Author

rgrigga commented Oct 3, 2014

i'm really sorry, this is a simple change I edited this inline, i don't know how to fix the tests.

@Zizaco Zizaco merged commit ace240b into Zizaco:master Oct 25, 2014
@Zizaco
Copy link
Owner

Zizaco commented Oct 25, 2014

@rgrigga Thanks

@boxfrommars boxfrommars mentioned this pull request Apr 1, 2015
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
4 participants
@rgrigga @coveralls @Zizaco @rgriss