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

Tweaks to maps per region #879

Merged
merged 3 commits into from
Jul 26, 2019
Merged

Conversation

cesswairimu
Copy link
Collaborator

Fixes #834

  • 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 22, 2019

Code Climate has analyzed commit d63002db and detected 5 issues on this pull request.

Here's the issue category breakdown:

Category Count
Complexity 1
Style 4

View more on Code Climate.

@codecov
Copy link

codecov bot commented Jul 22, 2019

Codecov Report

Merging #879 into main will increase coverage by 0.08%.
The diff coverage is 100%.

Impacted file tree graph

@@            Coverage Diff             @@
##             main     #879      +/-   ##
==========================================
+ Coverage   72.43%   72.51%   +0.08%     
==========================================
  Files          35       35              
  Lines        1335     1339       +4     
==========================================
+ Hits          967      971       +4     
  Misses        368      368
Impacted Files Coverage Δ
app/controllers/maps_controller.rb 91.89% <100%> (+0.14%) ⬆️
app/models/map.rb 93.19% <100%> (+0.09%) ⬆️

@jywarren
Copy link
Member

Cleared the CodeClimate issue! Is this ready for merge? Thanks!

@cesswairimu
Copy link
Collaborator Author

Yeah it is. Thanks @jywarren

@jywarren jywarren merged commit 47d4335 into publiclab:main Jul 26, 2019
@jywarren
Copy link
Member

🎉

chen-robert pushed a commit to chen-robert/mapknitter that referenced this pull request Dec 5, 2019
* cache regional maps

* add tag filter on maps

* test tag  filter in regional maps
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.

Tweaks to # of maps shown on front page, per "region"
2 participants