Enable `USE_THOUSANDS_SEPARATOR` by default & have some larger test ids about bakerydemo

[lb-]

Hmm. That's a good point. Do we know if this kind of issue will appear without that setting on?

Maybe a different approach is to run our Wagtail tests differently, instead of Bakerydemo data we update the data used in unit tests to be numbers above 1000 and run one version of the CI with this setting on?

[gasman]

My one reservation is that I'd rather not encourage greater use of USE_THOUSAND_SEPARATOR in the wild - IMO it really is a terrible design choice to apply formatting by default and require a template filter to turn it off, rather than the other way round - and if we put it in bakerydemo, people will copy it however much we advise against it. Maybe if we put a big capitalised PLEASE DON'T DO THIS comment next to it... :-)

[RashiJyotishi]

@lb-
Hi there! I understand the concern with the comma appearing before 3 digits due to the USE_THOUSAND_SEPARATOR setting. If you're looking for assistance in resolving this issue, I'm a beginner but eager to learn, and I'd be happy to give it a shot. Let me know if you'd like me to explore alternative solutions or make adjustments to the code. I'm here to help and keen on gaining more experience!