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.

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

Table: Make sparkline cell respect no value option. #75750

Merged
merged 1 commit into from Sep 29, 2023

Conversation

oscarkilhed
Copy link
Contributor

@oscarkilhed oscarkilhed commented Sep 29, 2023

What is this feature?

Let's the user set the no value field config option to display when a sparkline cannot be created from the data.

Why do we need this feature?

If there is no data or the data can't be used for a sparkline we currently get a static "no data". This makes the sparkline respect the no value option.

Fixes #75352

Special notes for your reviewer:

Please check that:

  • It works as expected from a user's perspective.
  • If this is a pre-GA feature, it is behind a feature toggle.
  • The docs are updated, and if this is a notable improvement, it's added to our What's New doc.

@oscarkilhed oscarkilhed requested review from a team, codeincarnate and mdvictor and removed request for a team September 29, 2023 14:00
@oscarkilhed oscarkilhed changed the title Make sparkline respect no value option Table: Make sparkline cell respect no value option. Sep 29, 2023
@grafana-delivery-bot grafana-delivery-bot bot added this to the 10.2.x milestone Sep 29, 2023
@oscarkilhed oscarkilhed added no-backport Skip backport of PR and removed area/frontend labels Sep 29, 2023
Copy link
Collaborator

@codeincarnate codeincarnate left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Nice, works for me!

@codeincarnate codeincarnate merged commit f39ed59 into main Sep 29, 2023
33 checks passed
@codeincarnate codeincarnate deleted the oscark/make-sparkline-respect-no-value branch September 29, 2023 14:54
rwwiv pushed a commit that referenced this pull request Oct 2, 2023
mildwonkey pushed a commit that referenced this pull request Oct 4, 2023
@zerok zerok modified the milestones: 10.2.x, 10.2.0 Oct 23, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Dashboards: Table panel using Cell type Sparkline does not take No value into account
3 participants