-
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
New Insights Charts #74
Conversation
…ture/insight-area-charts
…ture/insight-area-charts
✅ Deploy Preview for thingswedo ready!
To edit notification comments on pull requests, go to your Netlify site configuration. |
…ture/insight-area-charts
…ture/insight-area-charts
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.
pls refactor db manager and apply other changes (mainly deleting comments)
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.
Just some comments on database manager to go back to using generic seed method and seperate seed data from database methods
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.
Looks great!!
What type of PR is this? (check all applicable)
Description
Insights
Line Graph
New Stacked Area Chart
New Stream Graph
Other
Screenshots
Dynamic Ranges
Monthly Averages in Year View
Clipping Reduction
Area Chart
Stream Graph
UI accessibility checklist
If your PR includes UI changes, please utilize this checklist:
Critical
andSerious
issues?Added/updated tests?
Please aim to keep the code coverage percentage at 80% and above.
have not been included
Delete branch after merge?
What gif best describes this PR or how it makes you feel?