Cuong Do
Cuong Do
It looks like `call` was considered "ancient" as of 1 year ago: https://github.com/ethereum/solidity/issues/2884#issuecomment-329169020
cc @joshfraser
Possibly related intermittent failure: ``` 2) Marketplace Resource setNotification should allow notifications to be marked as read: Error: Timeout of 10000ms exceeded. For async tests and hooks, ensure "done()" is...
I haven't seen this locally since merging #486. I'm not sure what that'd be the case though.
Yes, this can only happen if someone is not using origin-js. I'm not putting a high priority on this issue, but I thought it best to at least record it.
I've sketched out what I'm thinking:  The contract I have so far is here: https://github.com/OriginProtocol/origin-js/blob/cuongdo/token_vesting/contracts/contracts/token/TokenVesting.sol Some tests are here: https://github.com/OriginProtocol/origin-js/blob/cuongdo/token_vesting/contracts/test/TestTokenVesting.js
@nick @joshfraser any comments?
Update: we've had some discussions, but nothing conclusive yet. Will continue to explore ideas but will postpone further implementation until mainnet launch is further along.
Depends on #304 and #305.
Assigning to @tyleryasaka, because he's already looking at the identity contracts