Skip to content
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.

Sign up for GitHub

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

Jump to bottom

add support for value mappings to stat panels. #223

Open
wants to merge 1 commit into
base: master
Choose a base branch
from
Open

add support for value mappings to stat panels. #223

wants to merge 1 commit into from

Conversation

AndersSoee
Copy link

Fixes #222

Requires PR for sdk lib : K-Phoen/sdk#22

Enables usage in the form:

		stat.ValueMappings(
			stat.RangeMapping(0, 0.9).Text("DOWN").Color("red"),
			stat.RangeMapping(1, 1).Text("UP"),
			stat.ValueMapping("UP").Color("green"),
		),

@codecov
Copy link

codecov bot commented May 2, 2023

Codecov Report

Merging #223 (15f8e87) into master (8a7600e) will increase coverage by 0.11%.
The diff coverage is 100.00%.

Additional details and impacted files

Impacted file tree graph

@@            Coverage Diff             @@
##           master     #223      +/-   ##
==========================================
+ Coverage   90.07%   90.18%   +0.11%     
==========================================
  Files          92       92              
  Lines        6934     7013      +79     
==========================================
+ Hits         6246     6325      +79     
  Misses        543      543              
  Partials      145      145
Impacted Files Coverage Δ
stat/stat.go 100.00% <100.00%> (ø)

@orloffv
Copy link

orloffv commented Mar 23, 2024

@K-Phoen, hi!
Grabana - great product, thank you.
Can you give feedback on this pull request, I need this feature

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

Support for range mappings on stat panel
2 participants
@AndersSoee @orloffv