Skip to content

Update to Django 5.2 LTS and Wagtail 7 LTS #4666

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.

Sign up for GitHub

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

Jump to bottom
Open
frjo wants to merge 22 commits into
base: main
Choose a base branch
from
Open

Update to Django 5.2 LTS and Wagtail 7 LTS #4666

frjo wants to merge 22 commits into from
+1,301 −1,406

Conversation

@frjo
Copy link
Member

@frjo frjo commented Jan 10, 2026 •
edited by wes-otf
Loading

Fixes #4250

These are big updated so everything needs to be tested.

Things to test:

  • The complete login procedure.
    • With enforce 2FA
    • For staff via Google
    • Without password
    • With password
    • Forgot password
    • Change password
  • Add and remove 2FA
  • The complete submission process for new and for old users.
  • The complete review process for reviewers and for staff.
  • Alla kind of file uploads and downloads. Do they work and do they have the correct permissions.
  • Permissions for all important views.
  • Notifications via slack, e-mail etc.

@frjo frjo added Type: Maintenance Type: Major Major change, used in release drafter labels Jan 10, 2026
@frjo frjo force-pushed the maintenance/django-5-wagtail-7 branch from 24133a9 to 625e107 Compare January 10, 2026 12:32
frjo
frjo commented Jan 10, 2026
View reviewed changes
hypha/apply/utils/blocks.py
# if child_data["type"] not in self.child_blocks:
# child_data["type"] = "char"

# return StreamValue(self, value, is_lazy=True)
Copy link
Member Author

@frjo frjo Jan 10, 2026

Choose a reason for hiding this comment

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

This needs investigation.

Copy link
Contributor

@wes-otf wes-otf Jan 15, 2026

Choose a reason for hiding this comment

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

@frjo so what you're saying is there was an change to StreamChild that broke this?

Copy link
Member Author

@frjo frjo Jan 16, 2026 •
edited
Loading

Choose a reason for hiding this comment

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

Yes, need to compare the overridden funktion in Wagtail 5 vs 6/7.

@frjo frjo changed the title Update to Django 5.2 LTS and Wagtail 7 LTS (via Wagtail 6 LTS) Update to Django 5.2 LTS and Wagtail 7 LTS Jan 11, 2026
@frjo frjo requested a review from wes-otf January 11, 2026 14:55
@wes-otf wes-otf added the Status: Needs dev testing 🧑‍💻 Tasks that should be tested by the dev team label Jan 13, 2026
@wes-otf
Copy link
Contributor

wes-otf commented Jan 15, 2026

so far everything has been working great & as expected!

frjo added 21 commits January 20, 2026 10:47
@frjo
Update to Wagtail 6.3.5 LTS as a first step.
0a1e6e4
@frjo
The wagtailadmin/shared/field_as_li.html template has been removed.
016d253
@frjo
The use_json_field attribute is no longer needed.
1727ce1
@frjo
Update wagtail admin custom user and groups views and templates.
0a1d9cb
@frjo
Disable to_python function in CustomFormFieldsBlock. REsult in TypeEr…
51b6ea8 
…ror but default function works.
@frjo
Add image field migration.
7db8390
@frjo
Update to Wagtail 7.x LTS.
f59dca8
@frjo
Add roles column to admin user list.
e2af732
@frjo
Update wagtail-factories.
061c331
@frjo
Add ignore_required_constraints to clean function.
056bd2d
@frjo
Updated to Django 5.2.10 LTS.
c8f7d7d
@frjo
Updated storage system for Django 5.
756f776
@frjo
Adjust storage settings.
8891a76
@frjo
Fix assertQuerySetEqual tests.
06e797b
@frjo
Fix tests: Logout must now be a post not get request. Use mark_safe o…
5df60b0 
…n SubmitButtonWidget.
@frjo
Adjust STORAGES settings.
0adb4e4
@frjo
Adjust STORAGES settings, revert.
43a8937
@frjo
Run collectstatic earlier in ci.
9eb1854
@frjo
Run collectstatic earlier in ci, part 2.
9b13533
@frjo
Fix STORAGES test setting.
5ca9467
@frjo
Fix setting FILE_FORM_TEMP_STORAGE after Django update.
01bcfa2
@frjo frjo force-pushed the maintenance/django-5-wagtail-7 branch from 65d951a to 01bcfa2 Compare January 20, 2026 09:50
@wes-otf
Copy link
Contributor

wes-otf commented Jan 21, 2026 •
edited
Loading

I found admin broke on the test instance until I deleted https://github.com/HyphaApp/hypha/blob/main/hypha/core/wagtail_hooks.py

it didn't locally for some reason though

@frjo
Copy link
Member Author

frjo commented Jan 22, 2026

Removed that file now. I removed all the wagtail css override files but missed this hook. It worked locally since the file was already in static.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@wes-otf wes-otf Awaiting requested review from wes-otf

Assignees

No one assigned

Labels

Status: Needs dev testing 🧑‍💻 Tasks that should be tested by the dev team Type: Maintenance Type: Major Major change, used in release drafter

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

Upgrade to Django 5.2 LTS

3 participants

@frjo @wes-otf