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

support multiple search fields #9

Closed
shalakhin opened this issue Apr 26, 2017 · 3 comments
Closed

support multiple search fields #9

shalakhin opened this issue Apr 26, 2017 · 3 comments

Comments

@shalakhin
Copy link
Collaborator

@and3rson I think it would be great if

widgets = {
    'user': SearchableSelect(model='users.User',
                             search_field='email',
                             many=False, limit=10),
}

will be

widgets = {
    'user': SearchableSelect(model='users.User',
                             search_field=['email', 'first_name', 'last_name'],
                             many=False, limit=10),
}

If you agree I will make it.

@shalakhin shalakhin self-assigned this Apr 26, 2017
@and3rson
Copy link
Owner

and3rson commented Apr 26, 2017

@shalakhin That's a great idea! We can definitely have it.
Also, don't forget to keep it backward-compatible with single string.

@shalakhin
Copy link
Collaborator Author

shalakhin commented Apr 26, 2017 via email

@and3rson
Copy link
Owner

@shalakhin seems like this can be done within the scope #10, I'll probably close this issue in favor of #10.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

2 participants