Skip to content
This repository was archived by the owner on Apr 12, 2024. It is now read-only.

revert: refactor($compile): remove preAssignBindingsEnabled leftovers #16595

Closed
wants to merge 1 commit into from

Conversation

gkalpak
Copy link
Member

@gkalpak gkalpak commented Jun 11, 2018

This reverts commit 8e104ee.

This internal clean-up turned out to break popular UI libraries (e.g.
ngMaterial, ui-bootstrap) and cause pain to developers.

Fixes #16594.

Verified

This commit was signed with the committer’s verified signature.
greg0ire Grégoire Paris
This reverts commit 8e104ee.

This internal clean-up turned out to break popular UI libraries (e.g.
`ngMaterial`, `ui-bootstrap`) and cause pain to developers.

Fixes angular#16594
Copy link
Member

@mgol mgol left a comment

Choose a reason for hiding this comment

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

I assume this is a straight-up revert of 8e104ee so LGTM as agreed.

@gkalpak gkalpak closed this in 42e622b Jun 11, 2018
@gkalpak gkalpak deleted the refactor-revert-16580 branch June 11, 2018 14:10
gkalpak added a commit that referenced this pull request Jun 11, 2018
This reverts commit 8e104ee.

This internal clean-up turned out to break popular UI libraries (e.g.
`ngMaterial`, `ui-bootstrap`) and cause pain to developers.

Fixes #16594

Closes #16595
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants