ReferenceError: moment is not defined

[volkerp]

moment.js functions do not work in current releases Chart.bundle.js and Chart.bundle.min.js.
moment.js has to be manually imported.

Regards

[simonbrunel]

@volkerp do you mean:

• a. Chart.bundle.*.js (2.7.1?) doesn't work (bug) or
• b. you can't use moment.js in your code?

If a. please provide way more details and a jsfiddle that reproduces your issue (as requested in our issue template displayed when creating an issue).

If b. the moment version embedded in Chart.bundle.*.js is private to Chart.js so if you want to use moment.js yourself, it's better to use Chart.*.js (non bundled) and import moment.js manually.

<script src="https://cdnjs.cloudflare.com/ajax/libs/moment.js/2.20.1/moment.min.js"></script>
<script src="https://cdnjs.cloudflare.com/ajax/libs/Chart.js/2.7.1/Chart.min.js"></script>

[mestaritonttu]

Test case that shows the error: https://codepen.io/anon/pen/odWRvZ

Copied from http://www.chartjs.org/samples/latest/scales/time/financial.html

[gdvalderrama]

@simonbrunel

the moment version embedded in Chart.bundle..js is private to Chart.js so if you want to use moment.js yourself, it's better to use Chart..js (non bundled) and import moment.js manually

It'd be nice to have this in the installation docs.

[simonbrunel]

@mestaritonttu that's case b, so you need to import moment yourself, separately to Chart.js (codepen).

@gdvalderrama agree, can you please submit a PR with that update in the "Bundled Build" section?

@etimberg am I correct about case b?

[simonbrunel]

I'm closing this ticket since we are not going to expose the internal moment. We are actually thinking to switch to luxon or date-fns for which I'm pretty sure we will not expose anything. IMO, we should even get rid of the confusing bundle version.

[gdvalderrama]

@simonbrunel sure, here's the PR: #5528