Thanks to visit codestin.com
Credit goes to github.com

No need to use user_id in recovery and settings controller #929

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

Open

adantart wants to merge 1 commit into dektrium:master from adantart:master

adantart commented May 23, 2017

Q	A
Is bugfix?	no
New feature?	yes
Breaks BC?	no
Fixed issues	#926


          No need to use user_id in recovery and settings controller

1bbeb37

Contributor

thyseus commented May 23, 2017

Good one. Could you re-open your PR here: https://github.com/dabilite/yii2-user/pulls ? Currently we have a fork of dektrium/yii2-user.

Member

thiagotalma commented Dec 27, 2017

Help is needed to review this.

@SamMousa @philippfrenzel @thyseus @bscheshirwork @tsdogs

thiagotalma added the status: to be verified label

thiagotalma force-pushed the master branch from 36d496a to 501de10 Compare

December 28, 2017 14:00

Contributor

bscheshirwork commented Jan 9, 2018

no need user_id. So clearly.

additional request to db
BC can be broken (url manager rule will be changed)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

status: to be verified