Skip to content

jqplots replaced by Chartjs#372

Merged
kevin-mcgoldrick merged 9 commits intomasterfrom
chartjs
Jan 31, 2025
Merged

jqplots replaced by Chartjs#372
kevin-mcgoldrick merged 9 commits intomasterfrom
chartjs

Conversation

@kevin-mcgoldrick
Copy link
Collaborator

jqplots replaced by Chartjs

  • primfaces has removed jqplots, this swapping the charting to the support chartjs pattern.
  • Resolves multiple security defects

jqplot

image

chartjs

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.

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.

Nice! LGTM

@kevin-mcgoldrick kevin-mcgoldrick merged commit 23d41f8 into master Jan 31, 2025
3 checks passed
@kevin-mcgoldrick kevin-mcgoldrick deleted the chartjs branch January 31, 2025 04:32
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