Skip to content
This repository has been archived by the owner on Aug 17, 2021. It is now read-only.

do scope apply after validate #139

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

pcamarajr
Copy link

@pcamarajr pcamarajr commented May 19, 2016

Added a scope.$apply inside a validate() function in act to update all bindings
Fixes #139

@mtrias
Copy link
Contributor

mtrias commented Jan 19, 2017

Not sure what issue you are trying to fix here, but sounds like this PR was inactive for a long time. Reopen if you still need this

@mtrias mtrias closed this Jan 19, 2017
@mtrias
Copy link
Contributor

mtrias commented Jan 19, 2017

Ok, found it. This is supposed to fix #138

@mtrias mtrias reopened this Jan 19, 2017
@TheSharpieOne
Copy link
Contributor

This can be closed along with the original issue, the 'working solution" doesn't actually do anything as scope.$apply is a variable referencing a function. This function is never executed since it doesn't get invoked using ()

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

Successfully merging this pull request may close these issues.

3 participants