Relax Account.name max_length restriction
jordanmkoncz opened this issue · 2 comments
jordanmkoncz commented
Currently, the Account.name
field is defined as name = models.CharField(max_length=50)
. Assuming there is no particular reason why the max_length
is restricted to 50 characters, could we increase this to max_length=255
?
In my case, I'm wanting to create some Accounts with names that include a UUID value (generated using uuid.uuid4
) which is 36 characters long by itself, making it hard to add other relevant content to the name
field.
adamcharnock commented
Hi Jordan,
Thanks for raising this. Yep, this can and should definitely happen. Do you
mind opening a PR with the model change and associated migration?
…On Tue, 18 Feb 2020 at 21:36, Jordan Koncz ***@***.***> wrote:
Currently, the Account.name field is defined as name =
models.CharField(max_length=50). Assuming there is no particular reason
why the max_length is restricted to 50 characters, could we increase this
to max_length=255?
In my case, I'm wanting to create some Accounts with names that include a
UUID value (generated using uuid.uuid4) which is 36 characters long by
itself, making it hard to add other relevant content to the name field.
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
<#47?email_source=notifications&email_token=AABBXZ2FSUUEHXK3UE2FCQTRDRID3A5CNFSM4KXM4JJKYY3PNVWWK3TUL52HS4DFUVEXG43VMWVGG33NNVSW45C7NFSM4IOOL7MA>,
or unsubscribe
<https://github.com/notifications/unsubscribe-auth/AABBXZ4NXZZ2SM32XEGMPXLRDRID3ANCNFSM4KXM4JJA>
.
adamcharnock commented
Four years later, and done!