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

fix($parse): Make sure ES6 object computed property to be watched (for master) #15678

Conversation

kindy
Copy link
Contributor

@kindy kindy commented Feb 4, 2017

this is same as #15637 but based on master branch.

Adding the missing watches for ES6 object property which added in #14407

What kind of change does this PR introduce? (Bug fix, feature, docs update, ...)
Bug fix

What is the current behavior? (You can also link to an open issue here)
$scope.$watch can not work as expected.

What is the new behavior (if this is a feature change)?
$scope.$watch works (NOT a feature change)

Does this PR introduce a breaking change?
NO

Please check if the PR fulfills these requirements

Other information:

Adding the missing watches for ES6 object property which added in angular#14407

(cherry picked from commit 4b8bbf8)
  (and also apply commit 7278762)
@gkalpak gkalpak closed this in 8162362 Feb 4, 2017
gkalpak pushed a commit that referenced this pull request Feb 4, 2017
Add the missing watches for ES6 object computed properties which were
implemented in #14407.

Closes #15678
@kindy
Copy link
Contributor Author

kindy commented Feb 6, 2017

@gkalpak
do we have any plan to apply this patch to v1.5.x ?

@gkalpak
Copy link
Member

gkalpak commented Feb 7, 2017

Since v1.6.x is the current stable version (and we are already working on v1.7.x), we do not plan to release any more 1.5.x versions (unless something critical comes up).

ellimist pushed a commit to ellimist/angular.js that referenced this pull request Mar 15, 2017
Add the missing watches for ES6 object computed properties which were
implemented in angular#14407.

Closes angular#15678
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.

3 participants