Skip to content

Conversation

@kdinev
Copy link
Member

@kdinev kdinev commented May 29, 2019

Closes #4908

@kdinev kdinev changed the title Ng 8 update b2 Updating dependencies to Angular 8 May 29, 2019
*@hidden
*/
@ContentChild(IgxDatePickerActionsDirective, { read: IgxDatePickerActionsDirective })
@ContentChild(IgxDatePickerActionsDirective, { read: IgxDatePickerActionsDirective, static: true })
Copy link
Member Author

Choose a reason for hiding this comment

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

@SAndreeva I'm pretty sure these should be static: false

*@hidden
*/
@ContentChild(IgxTimePickerActionsDirective, { read: IgxTimePickerActionsDirective })
@ContentChild(IgxTimePickerActionsDirective, { read: IgxTimePickerActionsDirective, static: true })
Copy link
Member Author

Choose a reason for hiding this comment

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

@SAndreeva I'm pretty sure these should be static: false

@ChronosSF
Copy link
Member

ChronosSF commented Jun 12, 2019

@kdinev , check #5058 before merging this one

ChronosSF and others added 2 commits June 13, 2019 10:07
@kdinev kdinev merged commit ab8479f into master Jun 13, 2019
@kdinev kdinev deleted the ng-8-update-b2 branch June 13, 2019 08:34
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.

Update dependencies to Angular 8