Skip to content

Migrate to Chart.js#415

Merged
kevin-mcgoldrick merged 3 commits intomasterfrom
chartjs
Aug 20, 2025
Merged

Migrate to Chart.js#415
kevin-mcgoldrick merged 3 commits intomasterfrom
chartjs

Conversation

@kevin-mcgoldrick
Copy link
Collaborator

@kevin-mcgoldrick kevin-mcgoldrick commented Aug 13, 2025

Utilize Chart.js because the next release of primefaces requires it

  • Replace primefaces chart model with Chart.JS
image

Please make sure these check boxes are checked before submitting

  • ** Squashed Commits **
  • ** All Tests Passed ** - mvn clean test -P default

** PR review process **

  • Requires one +1 from a reviewer
  • Repository owners will merge your PR once it is approved.

* master:
  Implement user auto-deletion follow up PR: Update TankConfig to account for XMLConfig changes (#417)
  Implement user auto-deletion for inactive accounts with configurable retention (default 2+ years) (#413)
Copy link
Collaborator

@Zakaria-Kofiro Zakaria-Kofiro left a comment

Choose a reason for hiding this comment

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

LGTM

@kevin-mcgoldrick kevin-mcgoldrick merged commit ce63b10 into master Aug 20, 2025
3 checks passed
@kevin-mcgoldrick kevin-mcgoldrick deleted the chartjs branch August 20, 2025 19:47
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.

2 participants