-
Notifications
You must be signed in to change notification settings - Fork 1
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
Dashboard Maps PoC #403
base: main
Are you sure you want to change the base?
Dashboard Maps PoC #403
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
quick self-review for a bit more context
@@ -0,0 +1,91 @@ | |||
SELECT |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
this is the original query. it isn't used for anything, i just didn't want to lose it.
@@ -0,0 +1,38 @@ | |||
SELECT |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
this query was modified from one on the superset instance to remove a table that wasn't there (the original query is below)
completed_work = CompletedWorkFactory( | ||
opportunity_access=opportunity_access, | ||
status=CompletedWorkStatus.approved, | ||
) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This is probably not right. But I don't think it matters at this stage.
{{ block.super }} | ||
{{ user_visits|json_script:"userVisits" }} | ||
<script type="module"> | ||
window.addEventListener('DOMContentLoaded', () => { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
the overwhelming majority of this is copy/pasted from here: https://docs.mapbox.com/mapbox-gl-js/example/cluster/
Just tweaked a few things to work with our data.
Context / Working Spec Here
This PR has a few pieces that I used to make a little map PoC in the dashboard:
This is the end result:
connect-map-poc.webm
Of course the data looks ridiculous when randomly generated, which is one reason it would be nice to have this deployed to the real app.