-
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
Fix tag search results for maps #5659
Conversation
@publiclab/reviewers |
It looks like the link to the word "here" (/maps) is broken due to:
--- Update: fixed this by clearing the cache |
Generated by 🚫 Danger |
@jywarren any idea why this is failing at post_test.rb? I'm not sure it has to do with my code and I saw you recently added that testing file |
Trying to fix a system test in #5661 - now we've resolved it, this may need a rebase? Thanks! |
7f85901
to
4ae0538
Compare
The test passed after rebase! thank you @jywarren |
Fixes #5658
(<=== Add issue number here)
Make sure these boxes are checked before your pull request (PR) is ready to be reviewed and merged. Thanks!
rake test
@publiclab/reviewers
for help, in a comment belowIf tests do fail, click on the red
X
to learn why by reading the logs.Please be sure you've reviewed our contribution guidelines at https://publiclab.org/contributing-to-public-lab-software
Thanks!
Screenshot of update: