-
Notifications
You must be signed in to change notification settings - Fork 0
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
29 new dashboard #30
29 new dashboard #30
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.
Amazing work @dgscwhite. All in two days having never used streamlit or git before 💪
I have added some suggestions and comments. Could you please make the changes I have suggested, or leave further comments. Once complete, fire me a message and I can resolve any open comments.
I will create separate issues for the following:
- Connect to the db in a function so we don't have reused code or exposed secrets
- Update the DB password since its in the git history here. I will let you know this
- To add more graphs
@mjennings061 many thanks for the positive feedback. All changes made and committed for review. |
@mjennings061 all done |
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.
User story complete.
New dashboard in streamlit with: