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

[machine] EthCfOpGenerator.update() shouldn't fail completely if no appID is provided #121

Closed
joelalejandro opened this issue Oct 24, 2018 · 0 comments
Labels
📦 Machine code related to packages/machine

Comments

@joelalejandro
Copy link
Contributor

public update(
message: InternalMessage,
context: Context,
cfState: CfState,
proposedUpdate: any
): CfOperation {
const multisig: Address = message.clientMessage.multisigAddress;
if (message.clientMessage.appId === undefined) {
// FIXME: handle more gracefully
throw Error("update message must have appId set");
}

@joelalejandro joelalejandro transferred this issue from another repository Oct 31, 2018
@joelalejandro joelalejandro added the 📦 Machine code related to packages/machine label Nov 1, 2018
snario added a commit to pawanjay176/contracts that referenced this issue Nov 6, 2018
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
📦 Machine code related to packages/machine
Projects
None yet
Development

No branches or pull requests

1 participant