Join collator pool: check that account is not already in collator pool

[prestonvanloon]

See TODO here:
https://github.com/prysmaticlabs/geth-sharding/blob/972509a8c2d3934cb6711e467138ac471b43b344/sharding/vmc.go#L58-L60

[terencechain]

the deposit method in SMC checks if account is already in collator pool, the following err catches it

tx, err := c.smc.SMCTransactor.Deposit(txOps)

[terencechain]

We can close this issue. SMC's deposit method checks if the account is in collator pool

[prestonvanloon]

The TODO still exists

[rauljordan]

Status on this issue? Can this be closed? @terenc3t @prestonvanloon

[terencechain]

we should close this, I don't see the TODO anymore

[rauljordan]

Perfect, feel free to close @terenc3t. 2018年4月23日(月) 10:23 terence tsao <notifications@github.com>:

we should close this, I don't see the TODO anymore — You are receiving this because you commented. Reply to this email directly, view it on GitHub <#39 (comment)>, or mute the thread <https://github.com/notifications/unsubscribe-auth/AFUIPQ5fnW4BuYDFfW1WK-2SroBeOmdCks5trg34gaJpZM4SSgiv> .

-- Sent from Mobile