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
fix(material): fix table generator schematic #16204
Conversation
Are there tests for schematics? |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
There should be tests for the generated code, but we don't have that set up yet
.../ng-generate/table/files/__path__/__name@dasherize@if-flat__/__name@dasherize__.component.ts
Outdated
Show resolved
Hide resolved
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM, thanks!
I removed the |
I think we should specify static false explicitly. So that the generated components can easily migrate to version 9 of Angular eventually. Sorry for the confusion. |
Import `MatTable` (used in `ViewChild` query). Add `static` option to view queries for future support of Angular version 9.
That should do it, Paul ☀ |
Import `MatTable` (used in `ViewChild` query). Add `static` option to view queries for future support of Angular version 9.
Import `MatTable` (used in `ViewChild` query). Add `static` option to view queries for future support of Angular version 9.
This issue has been automatically locked due to inactivity. Read more about our automatic conversation locking policy. This action has been performed automatically by a bot. |
Import
MatTable
(used inViewChild
query).Add
static
option to view queries for future support of Angular version 9.