Skip to content

#1046: add disableIncludeMissing to bar charts#1047

Open
samrichca wants to merge 2 commits intorelease-chartsfrom
include-missing
Open

#1046: add disableIncludeMissing to bar charts#1047
samrichca wants to merge 2 commits intorelease-chartsfrom
include-missing

Conversation

@samrichca
Copy link
Copy Markdown
Member

@samrichca samrichca commented Apr 1, 2026

Summary

Adds an optional property disableIncludeMissing to bar charts to hide the property "No Data".

Issues

Description of Changes

Charts

  • Add disableIncludeMissing to ChartsProvider to hide the "No Data" property.

Readiness Checklist

  • Self Review
    • I have performed a self review of code
    • I have run the application locally and manually tested the feature
    • I have checked all updates to correct typos and misspellings
  • Formatting
    • Code follows the project style guide
    • Automated code formatters (ie. Prettier) have been run
  • Local Testing
    • Successfully built all packages locally
    • Successfully ran all test suites, all unit and integration tests pass
  • Updated Tests
    • Unit and integration tests have been added that describe the bug that was fixed or the features that were added
  • Documentation
    • All new environment variables added to .env.schema file and documented in the README
    • All changes to server HTTP endpoints have open-api documentation
    • All new functions exported from their module have TSDoc comment documentation

Copy link
Copy Markdown
Member

@justincorrigible justincorrigible left a comment

Choose a reason for hiding this comment

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

LGTM

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.

2 participants