-
Notifications
You must be signed in to change notification settings - Fork 27
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
AdhocFilters: Add docs for AdhocFilterSet and AdhocFiltersVariable #377
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.
Did some wording and grammar edits. Mostly just removing passive voice.
Co-authored-by: Isabel <[email protected]>
Co-authored-by: Isabel <[email protected]>
Co-authored-by: Isabel <[email protected]>
Co-authored-by: Isabel <[email protected]>
Co-authored-by: Isabel <[email protected]>
Co-authored-by: Isabel <[email protected]>
Co-authored-by: Isabel <[email protected]>
Co-authored-by: Isabel <[email protected]>
Co-authored-by: Isabel <[email protected]>
Co-authored-by: Isabel <[email protected]>
Co-authored-by: Isabel <[email protected]>
Co-authored-by: Isabel <[email protected]>
Co-authored-by: Isabel <[email protected]>
Co-authored-by: Isabel <[email protected]>
Co-authored-by: Isabel <[email protected]>
Co-authored-by: Isabel <[email protected]>
Co-authored-by: Isabel <[email protected]>
Co-authored-by: Isabel <[email protected]>
// You don't need to set baseFilters, but they're useful if you want to limit label suggestions to only those you deem relevant for the scene. | ||
// These are not shown in the UI. | ||
baseFilters: [{ key: '__name__', operator: '=', value: 'ALERTS', condition: '' }], |
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.
An example would be useful for this property
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.
@dprokop the code your commenting on is the example (a very useful example of limiting keys to those of a specific metrics name)
Co-authored-by: Dominik Prokop <[email protected]>
Co-authored-by: Dominik Prokop <[email protected]>
Co-authored-by: Dominik Prokop <[email protected]>
Co-authored-by: Dominik Prokop <[email protected]>
🚀 PR was released in |
Docs!