Skip to content
This repository has been archived by the owner on Jul 6, 2022. It is now read-only.

Should voting use POLY instead of security token balances #52

Closed
adamdossa opened this issue Jan 16, 2018 · 3 comments
Closed

Should voting use POLY instead of security token balances #52

adamdossa opened this issue Jan 16, 2018 · 3 comments

Comments

@adamdossa
Copy link
Contributor

should contributedToSTO record the amount of POLY contributed, rather than the corresponding amount of security tokens issued? Since this mapping is used to vote, it may be better to use the POLY amount as this is harder to manipulate by the issuer (the issuer could issue a large number of security tokens to themselves for a small or zero amount of POLY and thus skew the vote). This would also require a corresponding change in voteToFreeze.

@satyamakgec
Copy link
Contributor

I agree with you, i will make the changes

@adamdossa
Copy link
Contributor Author

You need to also modify the voting function to divide by the correct amount (poly contributed not tokens issued).

@adamdossa adamdossa reopened this Jan 16, 2018
@satyamakgec
Copy link
Contributor

oops my bad

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

No branches or pull requests

2 participants