Charts: Update Victory dependencies

[mgrabovsky]

Please update the Victory dependencies to at least 36.5.1 as it fixes an annoying bug in build, see FormidableLabs/victory#2307.

Currently, I get a million warnings messages like the following when trying to build a Patternfly app:

Failed to parse source map from '/www/node_modules/victory-area/es/area.js.map' file: Error: ENOENT: no such file or directory, open '/www/node_modules/victory-area/es/area.js.map'

[nicolethoen]

@dlabrecq do you see any issues with this?

[dlabrecq]

Should be ok, but I'd like to test this with Cost Management. I had some trouble updating to the latest packages recently.

[dlabrecq]

There appears to be a lot of TypeScript changes for v36.5.1 and later. Looking at some of the changes in v36.5.3, I suspect there is more to come?

[dlabrecq]

Encountered a type error with Victory, which prevents me from building PatternFly. I was able to resolve the error, but it requires PatternFly to update to TypeScript v4.7.3

Created issue #7662

[dlabrecq]

I've updated our dependencies via PR #7674.

However, waiting for Victory to fix the TypeScript error above.

[stale]

This issue has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs.

[mgrabovsky]

@dlabrecq What is the current status here, please? It looks like #7674 was closed without merging with no follow-up.

[dlabrecq]

Unfortunately, I encountered a Type error that must be fixed in the Victory upstream. The animate prop should accept a boolean or object. It would be incorrect to change the type to accept a boolean only.

[dlabrecq]

I pushed a fix to the Victory upstream https://github.com/FormidableLabs/victory/pull/2464.
Waiting for the next release.

[dlabrecq]

This PR updates Victory dependencies to v36.6.7

#7986