Transfer Event To and From are Backwards.

[bh2smith]

When a user deposits GNO, the Transfer event of the LGNO token has recipient as "from" and contract as "to":

token-lock/packages/contracts/contracts/TokenLock.sol

Line 61 in 544a97d

emit Transfer(msg.sender, address(this), amount);

[rmeissner]

Same applies to the event in the withdraw method

[bh2smith]

The really unfortunate thing about this issue is that even with contract upgrade, all the historical events emitted by the contract prior to upgrade will forever have these fields reversed. Might even be more of an accounting headache to fix it and also messing up all the tax accounting software.