search

adamcharnock / django-hordak

Double entry accounting in Django
http://django-hordak.readthedocs.io
MIT License
254 stars 58 forks source link

Remove use of django-smalluuid #119

Closed adamcharnock closed 5 months ago

adamcharnock commented 6 months ago

django-smalluuid seemed useful when I wrote it, but now I no longer find it useful and it is weirdly non-standard. I'd like to get rid of it from Hordak.

DmytroLitvinov commented 5 months ago

Will be defualt uuid v4 used instead of maybe ulid?

adamcharnock commented 5 months ago

Supporting this seems very wise. The ideal is probably for this to be a user configurable callable, defaulting to uuid4() (which is included in Python core).

People could then configure this to use UUID7/ULID as they wish.

adamcharnock commented 5 months ago

Done. There is a new setting called HORDAK_UUID_DEFAULT, which is a callable. The default is uuid.uuid4()