airgradienthq / map

Other
8 stars 2 forks source link

Replace moment with luxon #7

Closed TomAdam closed 1 year ago

TomAdam commented 1 year ago

Moment is still a dependency due to chart.js requiring it. There is no reduction in bundle size, though this improves the handling of dates regardless and I think still worthwhile. See #5 for further details on where we could go next with this.

Note that this will likely crate a merge conflict with #6 due to both changing package.json and package-lock.json. I'm happy to rebase either if required.