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

Fix search bug #785

Merged
merged 3 commits into from
Jul 3, 2019
Merged

Fix search bug #785

merged 3 commits into from
Jul 3, 2019

Conversation

cesswairimu
Copy link
Collaborator

@cesswairimu cesswairimu commented Jul 2, 2019

Fixes #769

  • PR is descriptively titled 📑 and links the original issue above 🔗
  • tests pass -- look for a green checkbox ✔️ a few minutes after opening your PR -- or run tests locally with rake test
  • code is in uniquely-named feature branch and has no merge conflicts 📁
  • screenshots/GIFs are attached 📎 in case of UI updation
  • ask @publiclab/mapknitter-reviewers for help, in a comment below

@codeclimate
Copy link

codeclimate bot commented Jul 2, 2019

Code Climate has analyzed commit b5643c12 and detected 0 issues on this pull request.

View more on Code Climate.

@cesswairimu
Copy link
Collaborator Author

search

@codecov
Copy link

codecov bot commented Jul 3, 2019

Codecov Report

Merging #785 into main will increase coverage by 0.37%.
The diff coverage is 80%.

Impacted file tree graph

@@            Coverage Diff             @@
##             main     #785      +/-   ##
==========================================
+ Coverage   71.08%   71.46%   +0.37%     
==========================================
  Files          35       35              
  Lines        1335     1335              
==========================================
+ Hits          949      954       +5     
+ Misses        386      381       -5
Impacted Files Coverage Δ
app/helpers/front_ui_helper.rb 57.14% <0%> (+7.14%) ⬆️
app/controllers/maps_controller.rb 92.17% <100%> (+3.57%) ⬆️

@cesswairimu cesswairimu force-pushed the search-bug branch 3 times, most recently from 5285a9b to c5416b2 Compare July 3, 2019 11:42
@cesswairimu
Copy link
Collaborator Author

@publiclab/mapknitter-reviewers please review

@jywarren
Copy link
Member

jywarren commented Jul 3, 2019

Ah it was a failure with the rubocop linter!

@jywarren
Copy link
Member

jywarren commented Jul 3, 2019

Inspecting 28 files
.........................W..
Offenses:
app/helpers/front_ui_helper.rb:4:5: W: Lint/UselessAssignment: Useless assignment to variable - img. Use || instead of ||=.
    img ||= "/images/yarn.png"
    ^^^
28 files inspected, 1 offense detected

@jywarren jywarren merged commit cd7db17 into publiclab:main Jul 3, 2019
@jywarren
Copy link
Member

jywarren commented Jul 3, 2019

🎉 Great!!!

chen-robert pushed a commit to chen-robert/mapknitter that referenced this pull request Dec 5, 2019
* fix search bug

* test search template

* Update front_ui_helper.rb
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.

(Priority)Search Bug
2 participants