Add support for Wagtail-6

dinoperovic / django-salesman

Headless e-commerce framework for Django and Wagtail.

https://django-salesman.rtfd.io

BSD 3-Clause "New" or "Revised" License

384 stars 46 forks source link

Add support for Wagtail-6 #49

Closed vegaelle closed 5 days ago

vegaelle commented 1 month ago

This PR adds support for Wagtail >6.0, which removed wagtail.contrib.modeladmin, and recommends to use wagtail_modeladmin instead.

Tests still pass: poetry run pytest: :heavy_check_mark:
There are no lint errors: poetry run pre-commit run --all-files: :heavy_check_mark:

vegaelle commented 1 month ago

In order to fix failures with old Wagtail versions, i’ll add tests to use either wagtail.contrib.modeladmin or wagtail_modeladmin

dinoperovic commented 5 days ago

Hi @vegaelle, thank you for the PR, sorry I did not see it before.

This looks good, will merge and create a new release. Also adding you to the credits ;)

My plan is however to migrate to using Wagtail snippets for order display in the CMS, as that is the new way to show custom pages in Wagtail. This was something I started to work on, but haven't yet got the time to implement fully.

This is a good transition though, thanks!

dinoperovic commented 5 days ago

Version 1.2.1 was released.