Skip to content

Conversation

nickmoreton
Copy link
Contributor

@nickmoreton nickmoreton commented Aug 14, 2025

Do Not Merge

This is a test/experiment for using wagtail-orderable-viewset not with an official release yet.

Changes

  • The ProgrammeType model is no longer registered with wagtail-model-admin
  • A ModelViewSet is now used to register the model with Wagtail

Todo's

  • See if the wagtail-orderable-viewset code could be changed to avoid new migrations.

@nickmoreton nickmoreton changed the base branch from dev to support/wagtail-7 August 14, 2025 15:59
Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

While I can see this is required it wasn't the intention.

I needed to add the verbose_name & verbose_name_plural so the Title is shown on the reordering page.

I plan on looking the wagtail-orderable-viewset code base to try an avoid this.

]
list_display = [
"display_name",
"sort_order", # for reference with testing, not required
Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This column can be removed, I added it for debugging.

@nickmoreton nickmoreton changed the title Test wagtail orderable viewset Do Not Merge: WIP - Test wagtail orderable viewset Aug 14, 2025
Base automatically changed from support/wagtail-7 to master August 26, 2025 09:39
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant