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

AOT problem #84

Closed
wuzhouyang opened this issue Nov 23, 2016 · 8 comments
Closed

AOT problem #84

wuzhouyang opened this issue Nov 23, 2016 · 8 comments

Comments

@wuzhouyang
Copy link

i use DndModule.forRoot() in my app.module.ts . when i want to build the app with AOT, is throw an error :
image
so what can i do?

@wuzhouyang
Copy link
Author

wuzhouyang commented Nov 23, 2016

i think you need to add .metadata.json to support the AOT. look angular/angular#11262 (comment)
@akserg

@hansl
Copy link

hansl commented Dec 5, 2016

👍. You need to run ngc and publish the metadata.json file in your NPM package.

We had reports of this library not working with the Angular CLI, but unfortunately there's nothing we can do.

@rmcsharry
Copy link
Contributor

28 days ago this branch was created to fix AOT...is anyone able to merge it?

https://github.com/akserg/ng2-dnd/tree/aot

@akserg
Copy link
Owner

akserg commented Dec 23, 2016

Hi All,
Please, excuse me for the delay. The AOT support implemented in version 2.2.1 and tested in the following projects:

@rmcsharry
Copy link
Contributor

rmcsharry commented Dec 26, 2016

@akserg Thanks and Merry Christmas! I have tested 2.2.1 and the simple drag and drop works for AOT build.

I will test sortable dnd tomorrow.

@akserg
Copy link
Owner

akserg commented Dec 26, 2016

Hi @rmcsharry
Thank you for you help.

@rmcsharry
Copy link
Contributor

BTW, this is duplicate issue, why not close this one and continue to keep 69 open until more people confirm?

#69

@akserg
Copy link
Owner

akserg commented Dec 28, 2016

Please open the new issue if the problem still exists.

@akserg akserg closed this as completed Dec 28, 2016
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 a pull request may close this issue.

4 participants