-
Notifications
You must be signed in to change notification settings - Fork 24.8k
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(core): incorrectly validating properties on ng-content and ng-container #37773
fix(core): incorrectly validating properties on ng-content and ng-container #37773
Conversation
FYI, I've started a new presubmit (the previous one had some flakes). @crisbeto could you please also rebase this PR (to re-run CI with the most recent version of changes in master branch)? |
Rebased. |
d453a34
to
2c9a517
Compare
@crisbeto presubmit looks good, but the |
…tainer Fixes the following issues related to how we validate properties during JIT: - The invalid property warning was printing `null` as the node name for `ng-content`. The problem is that when generating a template from `ng-content` we weren't capturing the node name. - We weren't running property validation on `ng-container` at all. This used to be supported on ViewEngine and seems like an oversight. In the process of making these changes, I found and cleaned up a few places where we were passing in `LView` unnecessarily.
2c9a517
to
74c6722
Compare
Fixed the lint issue. |
…tainer (#37773) Fixes the following issues related to how we validate properties during JIT: - The invalid property warning was printing `null` as the node name for `ng-content`. The problem is that when generating a template from `ng-content` we weren't capturing the node name. - We weren't running property validation on `ng-container` at all. This used to be supported on ViewEngine and seems like an oversight. In the process of making these changes, I found and cleaned up a few places where we were passing in `LView` unnecessarily. PR Close #37773
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. |
…tainer (angular#37773) Fixes the following issues related to how we validate properties during JIT: - The invalid property warning was printing `null` as the node name for `ng-content`. The problem is that when generating a template from `ng-content` we weren't capturing the node name. - We weren't running property validation on `ng-container` at all. This used to be supported on ViewEngine and seems like an oversight. In the process of making these changes, I found and cleaned up a few places where we were passing in `LView` unnecessarily. PR Close angular#37773
Fixes the following issues related to how we validate properties during JIT:
null
as the node name forng-content
. The problem is that when generating a template fromng-content
we weren't capturing the node name.ng-container
at all. This used to be supported on ViewEngine and seems like an oversight.In the process of making these changes, I found and cleaned up a few places where we were passing in
LView
unnecessarily.