Skip to content

Update django-filter to 1.0 #144

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

Merged
merged 3 commits into from
Nov 17, 2016
Merged

Conversation

rpkilby
Copy link
Collaborator

@rpkilby rpkilby commented Nov 17, 2016

This makes drf-filters compatible w/ the 1.0 release of django-filter. Fixes #143.

Changes:

  • Adds AutoFilter as a common base class for RelatedFilter and AllLookupsFilter. The name reflects the filter auto-generation capabilities of the FilterSet w/ Meta.fields.
  • get_filters() => expand_filters(). This is a breaking change that is necessary due to the underlying API change in django-filter.
  • Moved import_class to utils
  • Made RelatedFilter compatible w/ the new get_queryset() on the underlying ModelChoiceFilter.
  • Refactored the metaclass to be compatible.

@rpkilby rpkilby added this to the v1.0.0 milestone Nov 17, 2016
@rpkilby rpkilby merged commit 7119eaa into philipn:master Nov 17, 2016
@rpkilby rpkilby deleted the update-django-filter branch November 17, 2016 18:55
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