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

Jbixon/map dashboard #81

Closed
wants to merge 7 commits into from
Closed

Jbixon/map dashboard #81

wants to merge 7 commits into from

Conversation

jbixon13
Copy link
Member

@jbixon13 jbixon13 commented Mar 30, 2020

Description

Adds mapping visualization to dashboard with dynamic popups. Color-coded markers not yet implemented.

Note: Test entities all have the same lat/long coordinates so it will look like only one popup is present. Change seed coordinates to fix this when testing.

Related PRs

List related PRs against other branches:

branch PR
other_pr_description #

Related Issues

List related Issues:

resolves #55

Todos

  • Tests
  • Documentation

Deploy Notes

Notes regarding deployment the contained body of work.

Steps to Test or Reproduce

Outline the steps to test or reproduce the PR here.

commands to test PR

Impacted Areas in Application

List general components of the application that this PR will affect:

  • Dashboard

@jbixon13 jbixon13 requested a review from stoopidJSON March 30, 2020 04:25
@stoopidJSON
Copy link
Member

Looks good, but we're going to hold this for phase 2 and build around it.

@stoopidJSON
Copy link
Member

This will have to be redone. The map lost priority due to city feedback. We'll circle back on this concept at a later date.

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.

Entity mapping
2 participants