Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Upgrade to Django 5.2 #1840

Open
3 tasks
ffont opened this issue Feb 12, 2025 · 1 comment
Open
3 tasks

Upgrade to Django 5.2 #1840

ffont opened this issue Feb 12, 2025 · 1 comment

Comments

@ffont
Copy link
Member

ffont commented Feb 12, 2025

We should upgrade to the next LTS version of Django (5.2)

Steps:

  • Clear Django 5 warnings (run tests with warnings option enabled to see them). This will require at least upgrading SESSION_SERIALIZER to django.contrib.sessions.serializers.JSONSerializer, which causes problems because we sometimes serialize Sound and File objects in the session. We need to reimplement the way we do that (or extend the serializer so it can serialize/unserialize these data types?)

  • Make sure Django dependencies are fine:

  • dj-database-url > should be fine with 5.x
  • django-admin-sortable > unsure about 5.x. We could look at django-admin-sortable2 ifneedbe.
  • django-amazon-ses > unsure about 5.x
  • django-cors-headers > should be fine with 5.x
  • django-debug-toolbar > should be fine with 5.x
  • django-extensions > should be fine with 5.x
  • django-multiupload > should be fine with 5.x
  • django-oauth-toolkit > should be fine with 5.x
  • django-object-actions > no notes about django compatibility, but latest version works with 4.2
  • django-ratelimit > no notes about django compatibility, but latest version works with 4.2
  • django-recaptcha > unsure about 5.x
  • django-redis > unsure about 5.x
  • django-silk > should be fine with 5.x
  • djangorestframework > should be fine with 5.x
  • sentry-sdk[django] > should be fine with 5.x

Potential follow-up steps:

@alastair
Copy link
Member

I vote that we remove django-redis. We only use it for basic django caching, and so therefore using the built-in caching backend will be fine

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants