-
Notifications
You must be signed in to change notification settings - Fork 22
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
Add a service to allow equivalent of search/wsuser #416
Comments
I'll add search to the windows app if you add it to the REST service. |
Any plans to implement this soon? |
No immediate plans PRs are welcome. |
Working on this soon. The biggest problem with the android app (warmshowers/wsandroid#73) really just cascades from the lack of this. |
Good timing for the WP app, I'm about to begin working on an update. |
Added hosts by_keyword in e3943ea |
This is deployed live, and is described in https://github.com/warmshowers/Warmshowers.org/wiki/Warmshowers-RESTful-Services-for-Mobile-Apps#bykeyword |
Thanks Randy! Will this return users who are currently unavailable? |
Yes, just like the warmshowers.org/search/wsuser, it can be used to search for any user. The currently available member of the array though lets you know what the status is. |
The Android app is using search/wsuser directly because it provides a simple search capability (full name, town, email) all in one string. However, the app is currently scraping to use it, which makes it fragile. (See warmshowers/wsandroid#69)
It should be easy to refactor and move the search/wsuser (hook_search) stuff into a place where it can be wrapped by a RESTful service, making this more robust.
wsuser_search is an invocation of hook_search(). The thing we have to do is wrap the query part so it's usable by both the service and hook_search.
The text was updated successfully, but these errors were encountered: