-
Notifications
You must be signed in to change notification settings - Fork 67
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
support for moment-timezone
#22
Comments
You can invoke like so: var momentTz = require('moment-timezone');
var moment = require('./moment-business-days')(momentTz); |
To me it seems this lib works with moment-timezone out of the box. |
It would be nice if |
I +1 this! |
How did you make it work? |
Any examples on how to make this work with ES6 import syntax? |
Does it count business day correctly when using moment-timezone? |
Awesome job @billforward-alex, thank you! |
This library is hardcoded to
require('moment');
— this makes it hard to coexist with other plugins.Better would be to expose an interface which lets the consumer supply a reference to
moment
.Sorry for not making a pull request, but here's what I did on my side (surrounded with a
module.exports = function(moment) {};
)…The text was updated successfully, but these errors were encountered: