-
Notifications
You must be signed in to change notification settings - Fork 1
Issues: StErMi/rareblocks-rent-poc
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.
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
Author
Label
Projects
Milestones
Assignee
Sort
Issues list
Invest further with tests if after doing need further investigation
stakers.remove
(EnumerableSet) that position will be empty or just directly removed
further investigation
#14
opened Feb 13, 2022 by
StErMi
updateAllowedSubscriptions should emit a AllowedSubscriptionsUpdate event and not for a single one
help wanted
Extra attention is needed
#12
opened Feb 13, 2022 by
StErMi
remove Extra attention is needed
indexed
keyword from onlyOwner events
help wanted
#11
opened Feb 13, 2022 by
StErMi
subscribe
function could useunchecked
for gas saving
wontfix
#10
opened Feb 13, 2022 by
StErMi
change Extra attention is needed
rareBlocksStaking
and newRareBlocksStaking
addresses as indexed param of relative events
help wanted
#9
opened Feb 13, 2022 by
StErMi
ensure that contract has enough balance to send it to staker contract adding check on address(this).balance >= stakingBalance
further investigation
need further investigation
#8
opened Feb 13, 2022 by
StErMi
Define the correct amount of days for subscription period
help wanted
Extra attention is needed
#7
opened Feb 13, 2022 by
StErMi
NOT_ENOUGH_FUNDS error message on subscribe could be confusing
help wanted
Extra attention is needed
#6
opened Feb 13, 2022 by
StErMi
setMaxSubscriptions should check newMaxSubscriptions != 0
help wanted
Extra attention is needed
#3
opened Feb 13, 2022 by
StErMi
ProTip!
Find all open issues with in progress development work with linked:pr.