-
-
Notifications
You must be signed in to change notification settings - Fork 137
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
Wagtail 6.3 updates #503
Wagtail 6.3 updates #503
Conversation
.github/workflows/test.yml
Outdated
@@ -11,7 +11,7 @@ jobs: | |||
runs-on: ubuntu-latest | |||
strategy: | |||
matrix: | |||
python: ["3.8", "3.9", "3.10", "3.11", "3.12"] | |||
python: ["3.9", "3.10", "3.11", "3.12"] |
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.
I suppose python 3.13 could be added now.
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.
OK thanks I can update this after the holidays.
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.
Python 3.13 added here: 9d8bf79
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.
I think this looks great. ❤️
Should we merge it or would you rather include Python 3.13 to the testing matrix first @nickmoreton ?
@MrCordeiro I think this is ready now. |
The changes primarily focus on updating the supported versions of Python, Django, and Wagtail, and removing support for older versions and adding some Wagtail version compatibility conditionals.
Updates to testing environment and compatibility:
.github/workflows/test.yml
: Removed support for Python 3.8 in the testing matrix.tox.ini
: Removed Python 3.8 and Django 3.2 from the environment list and added support for Wagtail 6.3 and Django 5.1. [1] [2]setup.py
: Removed Python 3.8 and Django 3.2 from the classifiers and updated thepython_requires
to '>=3.9'.Documentation updates:
README.rst
: Updated compatibility information to reflect the removal of Python 3.8 and Django 3.2, and the addition of Django 5.1.CHANGELOG.md
: Added an entry for the unreleased changes, including the updates to test Wagtail 6.3 and Django 5.1.Other updates:
I found that using Wagtail 6.3 the templates aren't the best UI and the option to delete a menu was no longer available, the delete button that was previously next to the save button was not available. The delete action link is now in the header bar drop down.
With wagtail 6.3 the snippets have the copy function available by default. As far as I could tell we don't need the copy function for a main menu so I have changed snippet viewset so that
copy_view_enabled = False