Re: Extending django.contrib.auth.models Group name to be 191 character max_length

2018-11-14 Thread Tim Allen
I've amended the PR to be 150 characters rather than 191. Thanks for doing the digging which explains why I had 191 characters stuck in my head. I'm in the "as long as possible to cover the most possible cases" camp, but 150 would seem to hit the 99.9%, so let's call it a win!  Regards,

Re: Extending django.contrib.auth.models Group name to be 191 character max_length

2018-11-14 Thread Carlton Gibson
(Before I start, I'm +1 but... ) Just having a quick go over the history here... * Original discussion floated a change to 100 or 150 chars. * Aymeric valued fixing the issue for 99.999% of cases. * Tom Christie wanted an "human value", not a machine one. 100 chars was suggested.