Skip to content

Conversation

jamesbiggs
Copy link
Contributor

This PR was made to satisfy this issue: #203

Simple PR that changes the SearchFields that have partial_match=True to use AutocompleteField and remove partial_match, which is bringing in a RemovedInWagtail60Warning when upgrading to wagtail 5.0.

Note: This is my first pull request/edit on an open-source repo. Sorry if I've missed anything or done something wrong - I'm open to learning! 😄

@jamesbiggs
Copy link
Contributor Author

Hey all, I've been using Wagtail for a few months now, and am fairly new to open-source code.

If I've missed any steps, or should/could do something differently - please let me know.

Hopefully these changes are okay!

Copy link
Member

@zerolab zerolab left a comment

Choose a reason for hiding this comment

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

LGTM. Thank you @jamesbiggs
Will look at making a new release tomorrow morning

@zerolab zerolab merged commit 35fffc4 into torchbox:main May 5, 2023
@zerolab
Copy link
Member

zerolab commented May 5, 2023

@jamesbiggs https://github.com/torchbox/wagtailmedia/releases/tag/v0.14.1

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.

2 participants