-
-
Notifications
You must be signed in to change notification settings - Fork 0
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
[Snyk] Upgrade recharts from 2.6.2 to 2.14.1 #49
base: main
Are you sure you want to change the base?
Conversation
Snyk has created this PR to upgrade recharts from 2.6.2 to 2.14.1. See this package in npm: recharts See this project in Snyk: https://app.snyk.io/org/khulnasoft-devsecops/project/4ffd4985-a266-4133-af54-bab5727b70a3?utm_source=github&utm_medium=referral&page=upgrade-pr
Reviewer's Guide by SourceryThis PR upgrades the Class diagram showing updated Recharts component changesclassDiagram
class RechartsComponents {
+onContextMenu()
+onDoubleClick()
}
RechartsComponents <|-- AreaChart
RechartsComponents <|-- LineChart
RechartsComponents <|-- BarChart
RechartsComponents <|-- ScatterChart
RechartsComponents <|-- PieChart
RechartsComponents <|-- RadarChart
RechartsComponents <|-- RadialBarChart
RechartsComponents <|-- FunnelChart
note for RechartsComponents "New event handlers added in v2.14.0"
class Axis {
-ref: removed
}
class PieSectorDataItem {
+payload: object
}
note for PieSectorDataItem "payload type changed from array to object"
State diagram showing React 19 compatibility changesstateDiagram-v2
[*] --> ReactCompatibility
ReactCompatibility --> DefaultPropsHandling: Update
DefaultPropsHandling --> ResponsiveContainer: Fix
ResponsiveContainer --> LegendComponent: Update
LegendComponent --> TypeScriptSupport: Enhance
TypeScriptSupport --> [*]
note right of ReactCompatibility: React 19 support added
note right of DefaultPropsHandling: Fixed defaultProps usage
note right of ResponsiveContainer: Improved element handling
note right of TypeScriptSupport: Enhanced type definitions
File-Level Changes
Tips and commandsInteracting with Sourcery
Customizing Your ExperienceAccess your dashboard to:
Getting Help
|
Here's the code health analysis summary for commits Analysis Summary
|
Important Review skippedIgnore keyword(s) in the title. Please check the settings in the CodeRabbit UI or the You can disable this status message by setting the Thank you for using CodeRabbit. We offer it for free to the OSS community and would appreciate your support in helping us grow. If you find it useful, would you consider giving us a shout-out on your favorite social media? 🪧 TipsChatThere are 3 ways to chat with CodeRabbit:
Note: Be mindful of the bot's finite context window. It's strongly recommended to break down tasks such as reading entire modules into smaller chunks. For a focused discussion, use review comments to chat about specific files and their changes, instead of using the PR comments. CodeRabbit Commands (Invoked using PR comments)
Other keywords and placeholders
CodeRabbit Configuration File (
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
We have skipped reviewing this pull request. It seems to have been created by a bot ('[Snyk]' found in title). We assume it knows what it's doing!
🎉 Snyk checks have passed. No issues have been found so far.✅ security/snyk check is complete. No issues have been found. (View Details) |
Snyk has created this PR to upgrade recharts from 2.6.2 to 2.14.1.
ℹ️ Keep your dependencies up-to-date. This makes it easier to fix existing vulnerabilities and to more quickly identify and fix newly disclosed vulnerabilities when they affect your project.
The recommended version is 36 versions ahead of your current version.
The recommended version was released 23 days ago.
Release notes
Package name: recharts
What's Changed
Fix
X/YAxis
: omit unused/accidentally added (in 2.13)ref
prop from axis types in #5295. Resolves #5294Full Changelog: v2.14.0...v2.14.1
What's Changed
Small release to get new top level chart events out there. Still working on 3.x, a little slower right now because of the holidays.
Feat
Area
/Line
/Bar
/Scatter
/Pie
/Radar
/RadialBar
/Funnel
Chart components: add support for top levelonContextMenu
andonDoubleClick
events #5254 by @ abarreau in #5255Fix
Typescript fixes
LabelList
:dataKey
prop for LabelList should accept any object by @ ckifer in #5252Pie
:PieSectorDataItem.payload
should be an object rather than an array by @ BTOdell in #5263New Contributors
Full Changelog: v2.13.3...v2.14.0
What's Changed
Small change which may (or may not) help when using ResponsiveContainer with React 19. Worth a shot.
!!
react-is
override is still needed for React 19 to work correctly with recharts !! - see #4558Fix
ResponsiveContainer
- swapisElement
fromreact-is
withReact.isValidElement
Full Changelog: v2.13.2...v2.13.3
What's Changed
ref
props back.Fix
Full Changelog: v2.13.1...v2.13.2
What's Changed
Fix
ResponsiveContainer
: fix internal bug from 2.13 which causes responsive container to render incorrectly by @ ckifer in #5174Bar
:radius
prop type fixed to match expected type by @ lifeofpavs in #5116New Contributors
Full Changelog: v2.13.0...v2.13.1
What's Changed
React 19 compat, fix annoying defaultProps warnings, bug fixes
Feat
React 19 compatibility - see #4558
peerDependencies
still cannot be updated until 3.0 release, but for now R19 beta/RC releases can be used with Recharts under the following conditions:react-is
version matches yourreact
andreact-dom
versions: use your package manager'soverride
functionality to match yourreact-is
version to your react version or else recharts elements will not show - see #4558 (comment)CartesianGrid
: add support forry
prop to curve the corner of the grid background - progress on #3062Fix
General
: FixdefaultProps
usage in function components, removes warnings on all affected components - see #3615ReferenceLine
: fix accidental removal of type annotation by @ zackliscio in #4610Line/Area
: fix issue whereclipDot
would not take effect if it was the only property indotProps
@ ckifer in #4674Tooltip
: fix issue where tooltip would not show in certain cases ifdata
was set on the graphical item by @ HHongSeungWoo in #4718X/YAxis
: use correct, adapted event types by @ ckifer in #4967Tooltip
:defaultIndex
out of range error by @ ckifer in #5054Pie
: fix duplicate key error @ ckifer in #4958Text
: fix duplicate key error by @ ckifer in #5088 closes #5072Typescript
New Contributors
Full Changelog: v2.12.7...v2.13.0
What's Changed
Some non-breaking bug fixes that need more testing with React 19. Fixes JSX type issue for React 19 when using
@ types/react@19
Fix
Line
/Area
:clipDot
setting fails to take effect if set as the only DotProp by @ ckifer in #4674 fixes #4671Tooltip
: Tooltip now renders correctly whendata
is set on the graphical item rather than the higher level chart by @ HHongSeungWoo in #4718 fixes #4717Pie
: fix duplicate key issue when "key" exists in chart data (try not use "key" as a data point) by @ ckifer in #4958 fixes #4858TypeScript
XAxis
/YAxis
: Correct Cartesian axis event types by @ ckifer in #4967 fixes #4959TypeScript
: return JSX in a function instead of a class component by @ ckifer in #4970 fixes React 19 type issues #4558 (comment)Full Changelog: v2.13.0-alpha.4...v2.13.0-alpha.5
What's Changed
Additional React 19 fixes - please keep adding reports to #4558, thanks!
Fix
ReferenceLine
: Re-add ReferenceLine types which caused a regression in alpha.0 by @ zackliscio in #4610New Contributors
Full Changelog: v2.13.0-alpha.3...v2.13.0-alpha.4
What's Changed
More React 19 specific fixes. If you are using R19, you must use
overrides
/resolutions
to match yourreact-is
version with your react versionFix
Full Changelog: v2.13.0-alpha.1...v2.13.0-alpha.3
What's Changed
v2.13.0-alpha.0 contains some fixes for React 19 support
v2.13.0-alpha.1 contains what seems to be the rest of them
Fix
Please test this as much as possible using React 16.8 - 19! thank you!
Full Changelog: v2.12.7...v2.13.0-alpha.1
Important
Note: You are seeing this because you or someone else with access to this repository has authorized Snyk to open upgrade PRs.
For more information:
Summary by Sourcery
Enhancements: