Skip to content
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

Bug - charts - victory version bump causing build errors in OCP #10403

Open
Tracked by #24
nicolethoen opened this issue May 21, 2024 · 0 comments
Open
Tracked by #24

Bug - charts - victory version bump causing build errors in OCP #10403

nicolethoen opened this issue May 21, 2024 · 0 comments
Assignees
Labels
Milestone

Comments

@nicolethoen
Copy link
Contributor

As soon as we bumped the version of victory in the current PF prerelease, OCP started getting the following errors in multiple places

I’m seeing the following error from a number of files/locations:

ERROR in ./node_modules/victory-cursor-container/es/victory-cursor-container.js 9:23
Module parse failed: Unexpected token (9:23)
You may need an appropriate loader to handle this file type, currently no loaders are configured to process this file. See https://webpack.js.org/concepts#loaders
|   // @ts-expect-error "TS2545: A mixin class must have a constructor with a single rest parameter of type 'any[]'."
|   return class VictoryCursorContainer extends Base {
>     static displayName = "VictoryCursorContainer";
|     static defaultProps = {
|       ...VictoryContainer.defaultProps,
 @ ./node_modules/victory-cursor-container/es/index.js 2:0-43 2:0-43
 @ ./node_modules/@patternfly/react-charts/dist/esm/components/ChartCursorContainer/ChartCursorContainer.js
 @ ./node_modules/@patternfly/react-charts/dist/esm/components/index.js
 @ ./node_modules/@patternfly/react-charts/dist/esm/index.js
 @ ./public/components/graphs/index.tsx
 @ ./public/components/graphs/helpers.ts
 @ ./public/actions/dashboards.ts
 @ ./public/reducers/dashboards.ts
 @ ./public/redux.ts
 @ ./public/components/app.jsx
 @ multi ./public/components/app.jsx monaco-editor-core/esm/vs/editor/editor.worker.js
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
Status: In Progress
Development

No branches or pull requests

3 participants