Skip to content
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

[api/change] Migrated to openwisp-users REST API filters #180 #181

Merged
merged 3 commits into from
Apr 21, 2023

Conversation

Aryamanz29
Copy link
Member

@Aryamanz29 Aryamanz29 commented Apr 21, 2023

Now that the multitenant django-filters PR has been merged (openwisp/openwisp-users#347), we can switch to using the multitenant version of django-filters in the API filters here.

Closes #180

@Aryamanz29 Aryamanz29 self-assigned this Apr 21, 2023
@coveralls
Copy link

coveralls commented Apr 21, 2023

Coverage Status

Coverage: 99.299% (+0.01%) from 99.289% when pulling 74c400a on switch-to-multitenant-dj-filters into 6c2d8a4 on master.

@Aryamanz29 Aryamanz29 marked this pull request as ready for review April 21, 2023 12:10
@Aryamanz29 Aryamanz29 requested a review from nemesifier April 21, 2023 12:10
@Aryamanz29 Aryamanz29 force-pushed the switch-to-multitenant-dj-filters branch from eeef644 to 28d67a6 Compare April 21, 2023 12:22
@Aryamanz29 Aryamanz29 changed the title [api/change] Switched to multitenant django-filters [api/change] Migrated to openwisp-users REST API filters #180 Apr 21, 2023
@Aryamanz29 Aryamanz29 force-pushed the switch-to-multitenant-dj-filters branch from 122fb07 to 74c400a Compare April 21, 2023 14:20
@nemesifier nemesifier merged commit 533dba3 into master Apr 21, 2023
@nemesifier nemesifier deleted the switch-to-multitenant-dj-filters branch April 21, 2023 19:10
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[change] Migrate to openwisp-users REST API filters
3 participants