-
Notifications
You must be signed in to change notification settings - Fork 1
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
Support/wagtail 42 upgrade #936
Conversation
14827a0
to
64cb594
Compare
Codecov Report
@@ Coverage Diff @@
## master #936 +/- ##
==========================================
- Coverage 52.99% 52.97% -0.02%
==========================================
Files 161 161
Lines 7688 7686 -2
Branches 164 164
==========================================
- Hits 4074 4072 -2
Misses 3497 3497
Partials 117 117
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@nickmoreton These changes all look good, and are complete as far as I can see. I've also done some testing of the site functionality, though I've not checked all the critical paths from the upgrading doc.
praise Thanks also for catching and addressing the changes which were still needed from previous upgrades.
Incl: pre-commit update
Now part of wagtail core
2a9c895
to
b68ca19
Compare
This upgrade wagtail to v4.2
Support ticket: 1166257309
Some of the changes here are not specifically for this upgrade and look like they are items form either version upgrades that were missed.
Additionally: