-
Notifications
You must be signed in to change notification settings - Fork 1.8k
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
(Link to) advanced search is wack! #800
Comments
I believe this is pending @ujithaperera's work in #781, but there could also be a separate issue in @david-days API code? Would you two mind taking a look? |
I will take a look. I am betting on routes.rb, which was reworked a few On Sep 7, 2016 5:56 PM, "Jeffrey Warren" notifications@github.com wrote:
|
This can be fixed easily by adding correct link to the page. But I also found that several route duplications in the the file. |
#810 was just merged in, and we'll be looking at advanced search functionality once it gets published on the site on Thursday; it may take some additional tweaks. |
This needs additional work for tests that return no results. I'll see what I can do! |
I wasn't able to reproduce this on publiclab.org and think perhaps it's been fixed already! |
What happened just before the problem occurred
i searched for a term for which there were no results. (i recreated this error with a neutral term "noresults" (haha): https://publiclab.org/search/noresults
Screenshot:
![screen shot 2016-09-07 at 5 22 42 pm](https://cloud.githubusercontent.com/assets/161439/18329045/c042edf6-751f-11e6-9c79-68d0dce45433.png)
On that page of no results, i clicked on the link in "Try an advanced search" -- which went to a broken page of URL https://publiclab.org/searches/advanced/
PublicLab.org username
liz
The text was updated successfully, but these errors were encountered: