search

torchbox / wagtail-footnotes

MIT License
20 stars 18 forks source link

[models] Switch default auto field to `BigIntegerField` #50

Closed olivierphi closed 1 year ago

olivierphi commented 1 year ago

Reaching the limits of a IntegerField takes some time, for sure... but when it happens it can be quite painful. :grimacing:

An example of this crashing the production environment is described there:

https://m.signalvnoise.com/update-on-basecamp-3-being-stuck-in-read-only-as-of-nov-8-922am-cst/

Maybe it could make sense to make sure that this Django app is not exposed to such an issue on very big Wagtail instances?

zerolab commented 1 year ago

Autofield was added in #38. This makes sense to me. I do want to see a project hitting that limit with wagtail-footnotes :D

olivierphi commented 1 year ago

Since #38 was merged, only awaiting to be part of a release, I'll close this MR :slightly_smiling_face: