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

Added ability to search for resources. #63

Merged
merged 1 commit into from
Sep 5, 2016

Conversation

msgerbush
Copy link
Contributor

This should close #20, I figure it's a little more flexible than the field filtering of DjangoFilterBackend, which requires exact matches. I added support for the basic 'name' fields on each model, but we can add more, if it makes sense.

  • Using REST Framework’s SearchFilter, added support for searching for
    resources.
  • Added docs explaining how to search and the fields on each resources
    that are searchable.
  • Added basic tests for using the search param.

Using REST Framework’s SearchFilter, added support for searching for 
resources.
Added docs explaining how to search and the fields on each resources 
that are searchable.
Added basic tests for using the search param.
@phalt
Copy link
Owner

phalt commented Sep 4, 2016

Wow, nice work @msgerbush ! Thanks for this - I'm going to look over it on Monday properly then I'll merge + ship :)

@phalt
Copy link
Owner

phalt commented Sep 5, 2016

@msgerbush will this require a migration?

@msgerbush
Copy link
Contributor Author

Nope, no migration!

@phalt
Copy link
Owner

phalt commented Sep 5, 2016

💃

@phalt phalt merged commit 1b75ca9 into phalt:master Sep 5, 2016
@phalt
Copy link
Owner

phalt commented Sep 5, 2016

@msgerbush deployed! It works really well

@msgerbush
Copy link
Contributor Author

@phalt, great! Thanks and glad to hear it's working well 😄

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.

Search for Resources via name
2 participants