Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Include "main" property in package.json #1285

Merged
merged 1 commit into from
Apr 6, 2015
Merged

Conversation

olanb7
Copy link
Contributor

@olanb7 olanb7 commented Mar 3, 2015

This will enable the datepicker to be included via require('bootstrap-datepicker') when using browserify/webpack.

This will enable the datepicker to be included via `require('bootstrap-datepicker')` when using browserify/webpack.
@panzerdp
Copy link

panzerdp commented Apr 6, 2015

It would be great to have this merged :) Thanks.

@acrobat
Copy link
Member

acrobat commented Apr 6, 2015

Yes this is something to merge soon! I'm not sure if main should be "main": "./dist/js/bootstrap-datepicker.js", or "main": "dist/js/bootstrap-datepicker.js". Or maybe that doesn't matter?

@panzerdp
Copy link

panzerdp commented Apr 6, 2015

@acrobat I think both options will work.

@acrobat
Copy link
Member

acrobat commented Apr 6, 2015

Ok thanks for the feedback @panzerdp! Thanks @olanb7 for the PR!

acrobat added a commit that referenced this pull request Apr 6, 2015
Include "main" property in package.json
@acrobat acrobat merged commit d18d270 into uxsolutions:master Apr 6, 2015
@acrobat acrobat added this to the 1.5.0 milestone Apr 6, 2015
@acrobat acrobat mentioned this pull request Apr 6, 2015
2 tasks
@olanb7 olanb7 deleted the patch-1 branch April 7, 2015 23:17
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants