Skip to content

Add Django 5.2 support#6

Merged
emily-bain merged 1 commit intosdelements:masterfrom
emily-bain:upgrade_to_django_5_2
Feb 26, 2026
Merged

Add Django 5.2 support#6
emily-bain merged 1 commit intosdelements:masterfrom
emily-bain:upgrade_to_django_5_2

Conversation

@emily-bain
Copy link
Copy Markdown
Member

  • Update tests to include 5.2 specific changes to accessibility attributes

Summary

  • Upgrading to Django 5.2.
  • Kept django-filter dependency to 25.1 to preserve Django 4.2 compatibility during upgrade process
Ready for QA Checklist
  • Code Review
  • Dev QA
  • Rebase and Squash

@emily-bain emily-bain force-pushed the upgrade_to_django_5_2 branch from cca4f3f to 5899e31 Compare February 25, 2026 19:29
* Update tests to include 5.2 specific changes to accessibility
  attributes
@emily-bain emily-bain force-pushed the upgrade_to_django_5_2 branch from 5899e31 to 34df922 Compare February 25, 2026 19:35
@Gee19 Gee19 self-requested a review February 26, 2026 20:55
@emily-bain
Copy link
Copy Markdown
Member Author

I haven't included a tag change in this PR yet, but re: tagging this with a version would the dependency upgrade make this a new 1.x release rather than a 1.1x release? This upgrade shouldn't strictly break Django 4.2 compatibility but I'm not sure off the top of my head if we're breaking 4.x compatibility before that in the diff.

@Gee19
Copy link
Copy Markdown
Member

Gee19 commented Feb 26, 2026

I think 1.1.0 makes sense regarding versioning, it's not a bug fix and it looks backwards compatible so no major version bump required imo 🤷

Copy link
Copy Markdown
Member

@Gee19 Gee19 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

lgtm +1

@emily-bain emily-bain merged commit 94bf813 into sdelements:master Feb 26, 2026
1 check passed
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

Successfully merging this pull request may close these issues.

2 participants