Thanks to visit codestin.com
Credit goes to github.com

Skip to content

Conversation

therbin404
Copy link

The js file was rewrited with the new javascript module templates

@pedrobaeza
Copy link
Member

/ocabot migration web_widget_json_graph

Please check pre-commit. You can install it locally for avoiding that problem.

@OCA-git-bot OCA-git-bot added this to the 16.0 milestone Apr 1, 2023
@OCA-git-bot OCA-git-bot mentioned this pull request Apr 1, 2023
47 tasks
@pedrobaeza
Copy link
Member

@pedrobaeza
Copy link
Member

The error is due to:

/home/runner/work/web/web/web_widget_json_graph/static/src/js/web_widget_json_graph.js
  4:1  error  Parsing error: 'import' and 'export' may appear only with 'sourceType: module'

If you are writing new-style JS, your file should be renamed to .esm.js.

@therbin404
Copy link
Author

I've disabled es-lint for the following line in web_widget_json_graph.esm.js :
this.chart = new Chart(this.canvasRef.el, this.data);

Chart is loaded by onWillStart(() => loadJS("/web/static/lib/Chart/Chart.js")); but es-lint does'nt seems to recognize it

@github-actions
Copy link

There hasn't been any activity on this pull request in the past 4 months, so it has been marked as stale and it will be closed automatically if no further activity occurs in the next 30 days.
If you want this PR to never become stale, please ask a PSC member to apply the "no stale" label.

@github-actions github-actions bot added the stale PR/Issue without recent activity, it'll be soon closed automatically. label Sep 21, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

stale PR/Issue without recent activity, it'll be soon closed automatically.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

6 participants