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

DM-1029: [App] Practice Page Geolocation Map #111

Merged
merged 3 commits into from Oct 31, 2019
Merged

Conversation

aurora-a-k-a-lightning
Copy link
Contributor

@aurora-a-k-a-lightning aurora-a-k-a-lightning commented Oct 30, 2019

JIRA issue link

https://agile6.atlassian.net/browse/DM-1029

Description - what does this code do?

Adds the geolocation feature to the practice page

Testing done - how did you test it/steps on how can another person can test it

Make sure to have the GOOGLE_API_KEY and run the diffusion_history task

  • browse to a practice page like FLOW3
  • scroll down to the Impact section
  • the map should be there
  • start playing with the filters

Screenshots, Gifs, Videos from application (if applicable)

image

Link to mock-ups/mock ups (image file if you have it) (if applicable)

https://www.figma.com/file/DDUSxYiD1MaWNrGFmMjY8d8a/Diffusion-Marketplace-Designs?node-id=4%3A306

Acceptance criteria

  • [ ]

Definition of done

  • Unit tests written (if applicable)
  • e2e/accessibility tests written (if applicable)
  • Events are logged appropriately
  • Documentation has been updated, if applicable
  • A link has been provided to the originating JIRA issue
  • No sensitive information (i.e. PII/credentials/internal URLs/etc.) is captured in logging, hardcoded, or specs

Copy link
Contributor

@nickclyde nickclyde left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good, great work!

@@ -126,6 +126,7 @@
<%= render partial: 'maps/home_map_data', formats: [:js] %>
<% end %>
<%= javascript_include_tag 'diffusion_history/home_map', 'data-turbolinks-track': 'reload' %>
<%= javascript_include_tag 'diffusion_history/_map_utilities', 'data-turbolinks-track': 'reload' %>
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Nice, good to move these to a separate file

expect(markers.count).to eq(17)
end
end
end
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Good test 👍

@aurora-a-k-a-lightning aurora-a-k-a-lightning merged commit 2922d08 into master Oct 31, 2019
aurora-a-k-a-lightning added a commit that referenced this pull request Mar 18, 2020
* DM-1029: start practice page map

* DM-1029: status filters and e2e tests
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.

None yet

2 participants