Skip to content

Conversation

ZPascal
Copy link
Owner

@ZPascal ZPascal commented Jan 29, 2022

TODO

  • Add Grafana V8 alerts support
  • Update the documentation and doc strings
  • Optimize the generated documentation
  • Integrate an integration test for alerting

UPDATE: Unfortunately, I need a few more days to finish the PR and follow up with the integration testing.

@ZPascal ZPascal changed the title Cw 4 add alerting Cw 4 + 5 add alerting and data source support Jan 29, 2022
@github-actions
Copy link
Contributor

github-actions bot commented Jan 29, 2022

Coverage Report
FileStmtsMissCoverMissing
src/grafana_api
   __init__.py0000
   alerting.py28151161
   alerting_notifications.py920380
   api.py350180
   dashboard.py1310560
   datasource.py1520660
   folder.py1110520
   legacy_alerting.py540260
   model.py770200
   search.py14161
tests/unittests
   test_alerting.py58401040
   test_alerting_notifications.py2150380
   test_api.py990180
   test_dashboard.py3220540
   test_datasource.py3580640
   test_folder.py2540360
   test_legacy_alerting.py1010180
   test_model.py25080
   test_search.py29060
TOTAL29346744 
Tests Skipped Failures Errors Time
258 0 💤 0 ❌ 0 🔥 2.232s ⏱️

…ionality

Add notification channel support
Add new functionality and first unittests
Add coverage badge and documentation
@ZPascal ZPascal closed this Feb 12, 2022
@ZPascal ZPascal deleted the CW-4-add-alerting branch February 12, 2022 10:27
@ZPascal ZPascal restored the CW-4-add-alerting branch February 12, 2022 10:27
@ZPascal ZPascal reopened this Feb 12, 2022
@ZPascal ZPascal marked this pull request as ready for review March 20, 2022 16:56
@ZPascal ZPascal merged commit 066a6d8 into main Mar 22, 2022
@ZPascal ZPascal deleted the CW-4-add-alerting branch March 22, 2022 06:47
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant