-
Notifications
You must be signed in to change notification settings - Fork 19
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
[Issue 845] ADR for measurement dashboard architecture #936
[Issue 845] ADR for measurement dashboard architecture #936
Conversation
Clones template and fills out initial sections
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.
Very thoughtful! I look forward to discussing.
documentation/decisions/adr/2023-12-18-measurement-dashboard-architecture.md
Show resolved
Hide resolved
documentation/decisions/adr/2023-12-18-measurement-dashboard-architecture.md
Show resolved
Hide resolved
documentation/decisions/adr/2023-12-18-measurement-dashboard-architecture.md
Outdated
Show resolved
Hide resolved
documentation/decisions/adr/2023-12-18-measurement-dashboard-architecture.md
Show resolved
Hide resolved
documentation/decisions/adr/2023-12-18-measurement-dashboard-architecture.md
Show resolved
Hide resolved
documentation/decisions/adr/2023-12-18-measurement-dashboard-architecture.md
Outdated
Show resolved
Hide resolved
documentation/decisions/adr/2023-12-18-measurement-dashboard-architecture.md
Outdated
Show resolved
Hide resolved
documentation/decisions/adr/2023-12-18-measurement-dashboard-architecture.md
Show resolved
Hide resolved
documentation/decisions/adr/2023-12-18-measurement-dashboard-architecture.md
Show resolved
Hide resolved
- Looker - Amazon Quicksight - Domo
- Updates note on API access to underlying data - Adds note about per-user cost
Adds this option and updates the bottom line tip for each option
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.
LGTM! Thanks for thinking this through so carefully!
documentation/decisions/adr/2023-12-18-measurement-dashboard-architecture.md
Show resolved
Hide resolved
|
||
## Pros and Cons of the Options <!-- OPTIONAL --> | ||
|
||
### S3 bucket + user interface |
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.
To clarify: would this used the component-based dashboard UI you were demo-ing the other day?
documentation/decisions/adr/2023-12-18-measurement-dashboard-architecture.md
Show resolved
Hide resolved
I don't really have any feedback here, looks good to me. |
documentation/decisions/adr/2023-12-18-measurement-dashboard-architecture.md
Show resolved
Hide resolved
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 such great work!!! My high level POV is that I think we should go straight for the "custom dashboard app" (eg. plotly-dash) or "open source dashboard" long term solution. Especially now that we have more infrastructure engineers.
@coilysiren Thanks for suggesting this! It's super helpful to know that y'all feel like you have a bit more capacity on the infra side to take on setting up a more robust data platform. Maybe the recommendation should be more of a parallel than a sequential approach? As described in the long-term recommendation, eventually we want both:
I see the s3 bucket + dashboard UI as a stepping stone to 2) with minimal up-front infra investment, so would love to work on that now. But if y'all have capacity, maybe we could also have you start building out the infrastructure for 1), which would involve a couple of decisions and outputs:
|
Summary
Creates an ADR that evaluates and recommends different architectural patterns we can use for our public measurement dashboard 30k deliverable (#65)
Fixes #845
Time to review: 10 mins
Changes proposed
Creates
documentation/decisions/adr/2023-12-18-measurement-dashboard-architecture.md
Context for reviewers
Updated the recommendation to the following:
Additional information