longbill/jquery-date-range-picker

Npm package wrong main file

Closed this issue · 4 comments

aaronkzhou commented

https://www.npmjs.com/package/jquery-date-range-picker

the main file should change from require('./src/jquery.daterangepicker.js') to require('./dist/jquery.daterangepicker.js');;

holtkamp commented

So what should it be?

require('./src/jquery.daterangepicker.js')

or

require('./dist/jquery.daterangepicker.js')

? Any documentation available on where to look this up?

aaronkzhou commented

it should require('./dist/jquery.daterangepicker.js');, it doesn't work for requiring require('./src/jquery.daterangepicker.js'), don't know why, just change the package.json main pls

holtkamp commented

Well, I do not have access to https://www.npmjs.com/package/jquery-date-range-picker, it seems https://www.npmjs.com/~youpinyao created it. @youpinyao?. The version there (1.0.4) also does not match "our" versions: https://github.com/longbill/jquery-date-range-picker/tags

So I am afraid I can not help...

holtkamp commented

This is probably resolved in https://github.com/longbill/jquery-date-range-picker/releases/tag/v0.19.0

Note we went for the 'src' approach. You should build it yourself, also see #441 (comment)