search

scottopolis / edd-metrics

EDD Metrics
46 stars 12 forks source link

Fix JS error in admin.js, add Webpack for building output. #49

Closed clifgriffin closed 4 years ago

clifgriffin commented 4 years ago

This does two things.

  1. There was a bug in admin.js that prevented the reports from loading fully when the date range was over 100 days. This is because lineChart is not included in the response object when the range is over 100 days.

  2. I went ahead and setup webpack. It builds the unminified/minified versions of both the JS and CSS in one step. You just run webpack or npm|yarn run build

I moved admin.js/admin.css into ./sources and then they output to ./dist. I included dist in git, as I assume this will make compiling a new version easier for release on .org.

Let me know if you have any questions.

scottopolis commented 4 years ago

Awesome, thanks Clif!