-
Notifications
You must be signed in to change notification settings - Fork 313
Commit
This commit does not belong to any branch on this repository, and may belong to a fork outside of the repository.
Main improved dashboard links (#622)
* Add missing options, functionality to DashboardLink Includes the ability to create direct links or a list of dashboard links, extra options like includeVars and targetBlank. * Add docstring to DashboardLink, remove the unused `dashboard` param Removing this parameter is a breaking change... It didn't actually impact the generated link, other than acting as an alias to 'title' in some cases. If preferred, I can restore it to retain that alias behaviour, and perhaps add a warning log message? * Update changelog * Fix the DashboardLink type param in docstrings * Fix the field name for URL in DashboardLinks * Correct type annotations for py37, py38 * Fix flake8 warnigns in docstring for DashboardLink * Add tests for DashboardLink * Fix flake8 complaints in tests/test_core.py
- Loading branch information
1 parent
f5da3c7
commit 6f36ce6
Showing
3 changed files
with
95 additions
and
12 deletions.
There are no files selected for viewing
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters