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

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!

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