-
Notifications
You must be signed in to change notification settings - Fork 435
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
Improve dashboard readme #1100
Improve dashboard readme #1100
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.
same feedback as @greghub, otherwise all good
@Traxmaxx thanks a lot! one more small comment, but pre-approved the changes |
@mlabouardy do you think we should include a link to a sample DB and config file to make it easier to run it for contributing? |
for sure, we can do it in a separate PR :) |
Problem
Readme was slightly outdated and infos where missing
Solution
Provided more test examples, updated reuirements and re-formatted the content.
Changes Made
How to Test
Screenshots
Notes
Checklist
Reviewers