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

Update UserToolComponent.php #42

Closed
wants to merge 1 commit into from
Closed

Conversation

Adnan0703
Copy link

behavior expects new password in 'password' not in 'new_password'

behavior expects new password in 'password' not in 'new_password'
@coveralls
Copy link

Coverage Status

Coverage remained the same at 43.051% when pulling 2353f0e on Adnan0703:patch-1 into 8675831 on burzum:master.

burzum pushed a commit that referenced this pull request Jan 24, 2017
@burzum
Copy link
Owner

burzum commented Jan 24, 2017

@Adnan0703 sorry for the late answer, I've missed this PR.

It should be made against the develop branch. Also to avoid issues when people (maybe even me) used new_password it should be just added not renamed.

I've fixed this in the develop branch and going to to a release later.

@burzum burzum closed this Jan 24, 2017
@Adnan0703 Adnan0703 deleted the patch-1 branch January 25, 2017 16:16
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