Give the boot to chartjs-adapter-moment

[kurkle]

No description provided.

[benmccann]

Wow! I'm not sure what you changed to get it working that I didn't figure out, but glad you figured it out!

We should update the docs too. E.g. integration.md says "The date adapter for Moment.js is included with Chart.js, but you still need to include Moment.js itself if this is the date adapter you choose to use.". Should update the migration guide too

You can also update some of the comments in the time scale code: benmccann@e586f4e

[benmccann]

I'm surprised you don't have to import chartjs-moment-adapter. How does it get pulled in and included?

[etimberg]

This may require website changes to ensure that the same files still work

[etimberg]

Looked closer. The samples will need to be updated as they do not include any adapter, only moment. Perhaps we can use this as an opportunity to convert them to use luxon instead

[benmccann]

It probably is a good idea to demonstrate something other than Moment. Luxon is more fully featured and has a better API and date-fns is smaller. Moment is rarely the best choice to use on the client side

[benmccann]

This is great! Thanks!