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

AMD Support #528

Closed
j4ys0n opened this issue Feb 5, 2015 · 6 comments
Closed

AMD Support #528

j4ys0n opened this issue Feb 5, 2015 · 6 comments
Labels

Comments

@j4ys0n
Copy link

j4ys0n commented Feb 5, 2015

Hello,

Could you please make this compatible with AMD API so that we can use this with Require?

Thanks!

@niklasvh niklasvh added this to the 0.5.0 milestone Feb 5, 2015
@niklasvh
Copy link
Owner

niklasvh commented Feb 5, 2015

Will get it added for 0.5.0 release

@j4ys0n
Copy link
Author

j4ys0n commented Feb 5, 2015

Awesome. Thank you!


Instagram: @jayssface
Twitter: @jayssface

On Thu, Feb 5, 2015 at 2:33 PM, Niklas von Hertzen
notifications@github.com wrote:

Will get it added for 0.5.0 release

Reply to this email directly or view it on GitHub:
#528 (comment)

@davidbau
Copy link

Hello @niklasvh - we'd like to use the library in an AMD application - looks like the 5.0 alpha doesn't yet support AMD, but maybe we'd add it as a patch. Any thoughts on the way you'd like to support it?

@xinan
Copy link

xinan commented Jul 1, 2015

Hi @niklasvh !
I realise that you are switching to use browserify. I am just curious about how would you make it AMD compatible when you use browserify? Thanks!

@kylebakerio
Copy link

What's the story on this? Require.js compatibility?

@niklasvh
Copy link
Owner

Should be fully supported now with the fixes in 4ebe9c5 and a4aa0c6

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

5 participants