- 
                Notifications
    You must be signed in to change notification settings 
- Fork 947
Upgrade to Wagtail 7 #16820
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
base: main
Are you sure you want to change the base?
Upgrade to Wagtail 7 #16820
Conversation
| Codecov Report✅ All modified and coverable lines are covered by tests. Additional details and impacted files@@            Coverage Diff             @@
##             main   #16820      +/-   ##
==========================================
- Coverage   79.69%   79.68%   -0.02%     
==========================================
  Files         159      159              
  Lines        8527     8525       -2     
==========================================
- Hits         6796     6793       -3     
- Misses       1731     1732       +1     ☔ View full report in Codecov by Sentry. 🚀 New features to boost your workflow:
 | 
| 
 @janbrasna does https://mozmeao.github.io/platform-docs/cms/#ive-downloaded-a-fresh-db-and-the-images-are-missing help ? | 
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.
| josepy==2.1.0 \ | ||
| --hash=sha256:0eadf09b96821bdae9a8b14145425cb9fe0bbee64c6fdfce3ccd4ceb7d7efbbd \ | ||
| --hash=sha256:9beafbaa107ec7128e6c21d86b2bc2aea2f590158e50aca972dca3753046091f | ||
| josepy==2.2.0 \ | 
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.
Also, I wanted to make a note here regarding #16534 that it's touching the josepy version here. In case that's something that needs to be watched in deployments, in case anything changes (and gets better locally or worse deployed).

One-line summary
This updates to use Wagtail 7.1.1 (the same as currently powering springfield)
Significant changes and points to review
These are the changes since last update work from Rob to LTS:
https://docs.wagtail.org/en/latest/releases/7.1.html
Issue / Bugzilla link
Closes #16334 as superseded.
Testing
yes please
not knowing too much about the internals I took it for a spin to see things work, and besides the usual images missing I hit all the content pages as:
http://localhost:8000/en-US/about/leadership/
http://localhost:8000/en-US/products/vpn/resource-center/
http://localhost:8000/en-US/products/vpn/resource-center/5-reasons-you-should-use-a-vpn/
http://localhost:8000/en-US/products/vpn/resource-center/what-is-a-vpn/
http://localhost:8000/en-US/products/vpn/resource-center/do-you-need-a-vpn-at-home-here-are-5-reasons-you-might/
http://localhost:8000/en-US/products/vpn/resource-center/what-does-a-vpn-do/
http://localhost:8000/en-US/products/monitor/
http://localhost:8000/en-US/products/monitor/what-is-a-data-broker/
http://localhost:8000/en-US/products/monitor/how-to-remove-your-information-from-the-internet/
etc. but please do take the CMS for a little spin to see nothing implodes with the models and types here.