Skip to content

feat: make the plugin ready for AoT compilation #30

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

Merged
merged 2 commits into from
Jan 16, 2017

Conversation

sis0k0
Copy link
Contributor

@sis0k0 sis0k0 commented Jan 16, 2017

  • add NgModule to export directiv
  • use ngc to generate metadata

implements #28

@sis0k0 sis0k0 force-pushed the aot-support branch 2 times, most recently from c90e4c8 to 7f91b9e Compare January 16, 2017 11:17
- add NgModule to export directiv
- use ngc to generate metadata
import * as frescoModule from "nativescript-fresco";
import * as applicationModule from "application";

import { registerElement } from "nativescript-angular/element-registry";

registerElement("FrescoDrawee", () => FrescoDrawee);
Copy link
Contributor

@VladimirAmiorkov VladimirAmiorkov Jan 16, 2017

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Shouldn't we do this in our TNSFrescoModule so that the users can simply add the module to the imports without requirement to register the element.

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Yep, you are right. It would be easier for the users that way 👍

@VladimirAmiorkov VladimirAmiorkov merged commit 934319b into master Jan 16, 2017
@sis0k0 sis0k0 deleted the aot-support branch January 16, 2017 12:07
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.

2 participants