Update Django to latest 4.2 LTS #2750 #2752
Conversation
Update Django from 3.2.23 to 4.2.7 via `poetry update` and pyproject.toml. ## Includes: - Move from django.utils.timezone utc to datetime.timezone.utc: RemovedInDjango50Warning - Remove to-be-deprecated USE_L10N setting, prior setting now enforced: RemovedInDjango50Warning - Add comment re Django 5.0 change in default USE_TZ value: Future default already adopted. - Comment STATICFILES_DIRS setting re build.sh & rockstor-jslibs. - Adopt new-in-Django-4.2 STORAGES setting to replace STATICFILES_STORAGE setting.
|
As per development details in linked draft PR, these changes depend upon the current testing channel of rockstor-jslibs as we have the following merged, but not yet in-release tar, PR dependency: |
|
Our rockstor-core PR test back-end fakes latest matched rockstor-jslibs repo branch, testing in this case, to allow for pre-tagged GitHub releases. Given this mechanism we have a successful rpmbuild and install of a test 5.0.5-2752 rpm (15.5 amd64 host), along with a preliminary Web-UI setup and basic/scant by-hand test under web browser development tools -> network -> disable cache to ensure we are using the rpm provided jslibs only. The resulting install also rebooted successfully with all core rockstor services starting as expected. |
|
@FroggyFlox & @Hooverdan96 I'll go ahead and merge this given the tests here and our unit tests executed during rpmbuild and the work done in the referenced draft PR. That jslibs issue was quite the surprise but we can re-approach that after we have our HTMX setup in place and are ready for Server Sent Events or the like. Plus we need this in place as part of our ongoing Python updates, and related pending Poetry update. |
Update Django from 3.2.23 to 4.2.7 via
poetry updateand pyproject.toml.Includes:
Fixes #2750
For development history see also draft PR #2751