Skip to content
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

Ignore spurious canExecute changes #49

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

Commits on May 1, 2015

  1. Ignore spurious canExecute changes

    canExecute is frequently set to a truthy value e.g. !isExecuting &&
    order() && order().amount() > 0. Knockout dependancy triggers updating
    this whenever any of the observables changes resulting in spurious
    updates to subscribers e.g. Activity.
    Coerce to a boolean to ignore spurious updates
    rockResolve committed May 1, 2015
    Configuration menu
    Copy the full SHA
    c354d82 View commit details
    Browse the repository at this point in the history