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.

Sign up for GitHub

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

Jump to bottom

[Chart.js] Build fails with latest 2.13.0 release #1262

Closed
DesLynx opened this issue Nov 9, 2023 · 3 comments · Fixed by #1263 or #1264
Closed

[Chart.js] Build fails with latest 2.13.0 release #1262

DesLynx opened this issue Nov 9, 2023 · 3 comments · Fixed by #1263 or #1264

Comments

@DesLynx
Copy link

DesLynx commented Nov 9, 2023

Hi!
I tried to update symfony/ux-chartjs to latest 2.13.0 release but then the webpack failed to compile...

yarn run v1.22.11
$ xxxxxxxxxxxxxxxxx/node_modules/.bin/encore dev
Running webpack ...

 ERROR  Failed to compile with 1 errors

Module build failed: Module not found:
"./node_modules/@symfony/ux-chartjs/dist/controller.js" contains a reference to the file "chart.js/auto".
This file can not be found, please check it for typos or update it if the file got moved.
@lyrixx
Copy link
Member

lyrixx commented Nov 9, 2023

Same issue here.

Reproducer:

symfony new  a --webapp
cd a
composer req encore
composer req symfony/stimulus-bundle
composer require symfony/ux-chartjs
yarn
yarn watch

@weaverryan
Copy link
Member

We're working on it :)

@smnandre smnandre mentioned this issue Nov 9, 2023
Merged
@weaverryan weaverryan mentioned this issue Nov 9, 2023
Merged
@DesLynx
Copy link
Author

DesLynx commented Nov 9, 2023

Thks everyone !

kbond added a commit that referenced this issue May 10, 2024
@kbond
bug #1263 [ChartJs] Replace chart.js/auto import with `Chart.regist…
796dd58 
…er` calls (smnandre)

This PR was merged into the 2.x branch.

Discussion
----------

[ChartJs] Replace `chart.js/auto` import with `Chart.register` calls

| Q             | A
| ------------- | ---
| Bug fix?      | yes
| New feature?  | no
| Issues        | Fix #1262
| License       | MIT

Ongoing attempt to fix ChartJS problems

Commits
-------

addc1f8 Fix Chart.js encore compilation
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
3 participants
@weaverryan @lyrixx @DesLynx