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

Decreased max_length to 150 on USERNAME_FIELD #30

Closed
wants to merge 2 commits into from

Conversation

ilhan
Copy link

@ilhan ilhan commented Apr 5, 2016

As Django does because of MySQL utfmb4 issue.

Django has experienced the same issue:
Django commit
Django developer mailing list discussion

Possible max_length seems to be 191 but Django chose 150.

…eld(....) as Django does because of MySQL utfmb4 issue
@coveralls
Copy link

Coverage Status

Coverage remained the same at 100.0% when pulling 4c58c86 on ilhan:less_max_length into cf2be14 on jcugat:master.

@ilhan
Copy link
Author

ilhan commented Apr 5, 2016

Sorry, didn't consider existing users and migrations. Skip this pull request, I keep this as my temporary fork. Also not sure how you want to handle migrations in the test folder.

@ilhan ilhan closed this Apr 5, 2016
@coveralls
Copy link

Coverage Status

Coverage remained the same at 100.0% when pulling 6289c9e on ilhan:less_max_length into cf2be14 on jcugat:master.

@ilhan
Copy link
Author

ilhan commented Apr 5, 2016

This is fixed in MySQL 5.7.7
Don't bother, time will solve this problem.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants