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.

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

Fixed loadUserByUsername method coding errors #5149

Closed
wants to merge 1 commit into from
Closed

Fixed loadUserByUsername method coding errors #5149

wants to merge 1 commit into from

Conversation

linasm83
Copy link

@linasm83 linasm83 commented Apr 5, 2015

Q A
Doc fix? yes
New docs? no
Applies to 2.6, 2.7, 3.0
Fixed tickets

@wouterj
Copy link
Member

wouterj commented Apr 7, 2015

Oew, I didn't need to think a lot about the correctness of this PR :) Thanks Linas! I've merged it into the 2.3 branch, as the issues exists there as well. That's why you'll see a close status on GitHub instead of merged. Anyway, it's included in the docs now.

wouterj added a commit that referenced this pull request Apr 7, 2015
…icius)

This PR was submitted for the 2.6 branch but it was merged into the 2.3 branch instead (closes #5149).

Discussion
----------

Fixed loadUserByUsername method coding errors

| Q             | A
| ------------- | ---
| Doc fix?      | yes
| New docs?     | no
| Applies to    | 2.6, 2.7, 3.0
| Fixed tickets |

Commits
-------

3a7acfb Fixed loadUserByUsername function coding errors
@wouterj wouterj closed this Apr 7, 2015
@linasm83
Copy link
Author

linasm83 commented Apr 7, 2015

Will be a lesson for me to check a bit different next time, in 2.4 and 2.5 code was different and correct, didn't thought at it will have the same issue in 2.3 :)

@wouterj
Copy link
Member

wouterj commented Apr 7, 2015

The 2.4 and 2.5 versions are no longer maintained (we only maintain versions that haven't reached end of life)

@linasm83 linasm83 deleted the fix_user_provider_code branch April 10, 2015 20:59
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants