This repository has been archived by the owner on Dec 13, 2019. It is now read-only.

[contracts] Update `setState` method to work on non-settled apps vs just "ON" state apps #1411

Open

snario opened this issue May 6, 2019 · 0 comments

Labels

📦 Contracts 💎 Ethereum good first issue

Contributor

snario commented May 6, 2019

There is a bug in the setState function which checks that the app is ON. See

monorepo/packages/contracts/contracts/mixins/MixinSetState.sol

Line 45 in 99fc555

"setState was called on an app that is either in DISPUTE or OFF"

It should instead check if the app is either ON or DISPUTE (but not yet timed out).

The text was updated successfully, but these errors were encountered:

snario added 📦 Contracts good first issue 💎 Ethereum labels

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