Should be isEnabled - Githubissues

VeeamHub / grafana

Grafana dashboard for Veeam solutions

MIT License

59 stars 19 forks source link

Should be isEnabled #12

Closed TnTBass closed 3 years ago

TnTBass commented 3 years ago

We would rather store the enabled status, not the name ;)

Type of change

[X] Bug fix (non-breaking change which fixes an issue)
[ ] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to not work as expected)
[ ] This change requires a documentation update

How Has This Been Tested?

Tested in my environment

Checklist (check all applicable):

[X] My code follows the style guidelines of this project
[X] I have performed a self-review of my own code
[X] I have commented my code, particularly in hard to understand areas
[X] I have made corresponding changes to the documentation
[X] My changes generate no new warnings
[X] I have added tests that prove my fix is effective or that my feature works
[X] New and existing unit tests pass locally with my changes