Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

comment ambiguity about unit of totalCFOEarnings and totalLostAndFoundBalance #20

Open
sunsetlover opened this issue Nov 16, 2018 · 2 comments
Open

comment ambiguity about unit of totalCFOEarnings and totalLostAndFoundBalance #20

sunsetlover opened this issue Nov 16, 2018 · 2 comments
Assignees
@hwrdtm

Comments

@sunsetlover
Copy link

The comment on lines 14 and 16 in OffersAccessControl.sol is ambiguous about the unit of totalCFOEarnings and totalLostAndFoundBalance which I believe is in wei.
@hwrdtm
Copy link
Contributor

hwrdtm commented Nov 16, 2018

Thanks for pointing it out @sunsetlover! We'll make the comments more explicit 👍

@hwrdtm hwrdtm self-assigned this Nov 16, 2018
@jordankitty
Copy link
Contributor

Thanks for your participation, @sunsetlover! Our team has reviewed your submission, and while we are grateful, we have decided it does not warrant a reward.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@hwrdtm hwrdtm

Labels
None yet
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
3 participants
@hwrdtm @jordankitty @sunsetlover