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

Multisignature transaction sets property with incorrect casing for multiMin and multiLifetime when applying/undoing transaction. #1140

Closed
SargeKhan opened this issue Feb 25, 2019 · 1 comment

Comments

@SargeKhan
Copy link
Contributor

Expected behavior

The properties defined for account related to multisignatures should be in camel case.

Actual behavior

The multisignature transaction assigns these properties to account in lower case.

Steps to reproduce

Compare code with core.

Which version(s) does this affect? (Environment, OS, etc...)

2.1.0

@shuse2
Copy link
Contributor

shuse2 commented Feb 25, 2019

It should update to

readonly multiMin?: number;
readonly multiLifetime?: number;

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

No branches or pull requests

2 participants