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

Compile error with fullTemplateTypeCheck: true #84

Closed
achimha opened this issue Jan 16, 2018 · 5 comments
Closed

Compile error with fullTemplateTypeCheck: true #84

achimha opened this issue Jan 16, 2018 · 5 comments

Comments

@achimha
Copy link
Contributor

achimha commented Jan 16, 2018

I'm submitting a ... (check one with "x")

[X] bug report => search github for a similar issue or PR before submitting

Current behavior
Angular 5.2 introduces a new AoT compiler option to perform a more thorough type check on templates. It is expected that this option will default to true in a future version:

https://blog.angular.io/angular-5-2-now-available-312d1099bd81

Building a project with ng2-archwizard shows one type error:

node_modules/ng2-archwizard/dist/components/wizard-navigation-bar.component.d.ts.WizardNavigationBarComponent.html(17,11): : Directive GoToStepDirective, Expected 0 arguments, but got 1.

  • ng2-archwizard version: 2.1.0

  • Angular version: 5.2.0

  • Browser: [all ]

  • Language: [TypeScript 2.5.2]

@madoar
Copy link
Owner

madoar commented Jan 16, 2018

Thank you for testing the library with angular 5.2!

Taking a quick look at the source code I'm not sure where the error is located.
The message points to both the html template of WizardNavigationBarComponent and to GoToStepDirective, but I'm not sure where exactly typescript/angular expects 0 arguments but gets 1.

Sadly I have no test project using angular 5.2, so it's hard for me to test this.
Can you provide a short test project using angular 5.2 with which I can experiment to find and remove the error?

@achimha
Copy link
Contributor Author

achimha commented Jan 17, 2018

The same problem might exist in other directives, sorry for the incomplete patch.

@madoar
Copy link
Owner

madoar commented Jan 19, 2018

@achimha no problem. Thanks for your effort!
Today I've pushed ng2-archwizard to angular 5 and prepared the project for a new development iteration (i.e. version 3.0.0).
I hope you can merge/rebase the current master branch in your PRs to ensure that the tests run correctly and everything is compatible.
Afterwards I would love to merge them :)

@achimha
Copy link
Contributor Author

achimha commented Jan 20, 2018

Created a new PR #89

@madoar
Copy link
Owner

madoar commented Jan 23, 2018

This was fixed in #89.

@madoar madoar closed this as completed Jan 23, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

2 participants