Skip to content

Kien.luu/always create chart if has no data#724

Closed
trungkien1208 wants to merge 4 commits into
jtblin:mainfrom
taggle-health:kien.luu/always-create-chart-if-has-no-data
Closed

Kien.luu/always create chart if has no data#724
trungkien1208 wants to merge 4 commits into
jtblin:mainfrom
taggle-health:kien.luu/always-create-chart-if-has-no-data

Conversation

@trungkien1208
Copy link
Copy Markdown
Contributor

Description of change

Pull Request check-list

  • Run gulp test to ensure there are no linting, or style issues and all tests pass.
  • Squash your commits into a few commits only.
  • Make sure the commit message is short, concise and descriptive of the issues you're fixing.
  • Avoid mixing up multiple issues and/or features, open one pull request for each issue.
  • Have you updated the documentation and / or examples?
  • Have you included a new test?

@trungkien1208 trungkien1208 force-pushed the kien.luu/always-create-chart-if-has-no-data branch from 55817ba to 892b982 Compare August 4, 2021 07:46
@jtblin jtblin deleted the branch jtblin:main April 29, 2026 05:26
@jtblin jtblin closed this Apr 29, 2026
@jtblin jtblin reopened this Apr 29, 2026
@jtblin jtblin changed the base branch from master to main April 29, 2026 10:41
jtblin added a commit that referenced this pull request Apr 29, 2026
- Added chart-plugins support (#643)
- Added chart-display-when-no-data and chart-force-update (#724)
- Implemented dynamic chart-options updates (#724)

Co-authored-by: BHARATH CHEKURI <bchekuri@outlook.com>
Co-authored-by: Kien Luu <luutrungkien120894@gmail.com>
@jtblin
Copy link
Copy Markdown
Owner

jtblin commented Apr 29, 2026

I have implemented these enhancements in 689fd84. I've manually merged the logic and refactored it to support dynamic options updates as well.

I've included you as a co-author in the commit history to ensure you get full credit and appear in the project's contributor list.

Thanks for the contribution!

@jtblin jtblin closed this Apr 29, 2026
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.

3 participants