-
Notifications
You must be signed in to change notification settings - Fork 258
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
webui: analytics module: show stored data per jobname in treemap #1082
webui: analytics module: show stored data per jobname in treemap #1082
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.
Good work.
- Do we want/need to have some docs about the new analytics module?
- The CHANGELOG.md commit is missing.
I don't think so, because the treemap is actually self-explanatory for users.
I'll add it after rebase and solving merge conflicts due to some pending PRs that will be merged first, I guess. |
After a second thought and look about docs, we should add the new module here https://docs.bareos.org/IntroductionAndTutorial/BareosWebui.html#command-acl-requirements due to command.csv changes. Going to add an extra commit for that. |
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.
Good work!
This PR introduces an analytics module into webui. The first feature of this new module
is a visualization of stored backup job data in a treemap.
Thank you for contributing to the Bareos Project!
Please check
If you have any questions or problems, please give a comment in the PR.
Helpful documentation and best practices
Checklist for the reviewer of the PR (will be processed by the Bareos team)
General
Source code quality
bareos-check-sources --since-merge
does not report any problemsgit status
should not report modifications in the source tree after building and testing