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

Fix/top up allowed tokens update #76

Open
wants to merge 296 commits into
base: master
Choose a base branch
from
Open

Fix/top up allowed tokens update #76

wants to merge 296 commits into from

Conversation

zuzueeka
Copy link

No description provided.

zuzueeka and others added 30 commits September 12, 2022 20:08
@zuzueeka
fix: mismatched code style for _easy_track arg in constructor
4148ab0
@zuzueeka
fix: added check for zero periodDuration
583dac4
@zuzueeka
deleted test_allowed_recipients_happy_path since all events from it a…
2ef116e 
…re covered by other tests
@zuzueeka
fix: made the calculation of the start date of the period more clear.…
90675dc 
… added tests for calculating the number of the first month in a period.
@zuzueeka
fix: added ganache in brownie-config.yaml and in devDependencies in p…
1ab8c3d 
…ackage.json
@zuzueeka
fix: transferred getting the number of the first month in the period …
2ee835a 
…to Wrapper
@zuzueeka
fix: changed the types of variables
e8d61bc
@zuzueeka
fix: added a check to avoid math underflow error
1d289cc
@arwer13
ref(test, allowed recipients): better variable names
5306c88
@arwer13
test(allowed recipients): change limits when top-up is in flight
21103e4
@arwer13
tests(limits): one more bunch of tests
eecf2b7
@arwer13
ref(limit tests): mostly cleaning up
fb91cd1
@arwer13
add(test limits): check LimitsChecker period range
fad0ce7
@zuzueeka
fix: contracts/AllowedRecipientsRegistry.sol: formatting, made not de…
c4b3f38 
…rive from AccessControl, renamed methods
@zuzueeka
fix: contracts/AllowedRecipientsRegistry.sol: formatting, made not de…
cdc3f53 
…rive from AccessControl, renamed methods
@zuzueeka
fix: renaming
669c33e
@zuzueeka
fix: renaming
f7bc923
@zuzueeka
fix: contracts/EVMScriptFactories/TopUpAllowedRecipients.sol: comment…
47e1039 
…s, renaming
@zuzueeka
fix: contracts/LimitsChecker.sol: comments, renaming, typos
f5630e3
@zuzueeka
fix: contracts/LimitsChecker.sol: comments, renaming, typos
7b44cce
@arwer13
ref(tests): shorten access_control_revert_message name
1c38969
@arwer13
test(limits): allowed recipients registry roles
4adc71d
@zuzueeka
fix: LimitsChecker.sol: comments, renaming, formatting, put things in…
199b300 
… order in methods
@arwer13
fix(LimitsChecker): typo in function name
95f54c8
@zuzueeka
fix: comments, renaming, formatting
730f034
@TheDZhon
fix: deploy script for allowed recipient
8911e0c
@zuzueeka
fix (limitsChecker): currentSpendableBalance -> spendableBalance
3afe0ff
@zuzueeka
fix (limitsChecker): getCurrentPeriodState -> getPeriodState
f74802b
@zuzueeka
fix: renaming scripts/deploy_limited_allowed_recipients.py -> deploy_…
8a5654b 
…limit_enabled_payout_factories
@zuzueeka
fix: chenges in README
9e78e85
zuzueeka and others added 30 commits May 22, 2023 21:36
@zuzueeka
add condition for steth error margin check
addd68d
@zuzueeka
rename STETH_ERROR_MARGIN to STETH_ERROR_MARGIN_WEI
8c7d4a5
@zuzueeka
add shares balances checks
91b907f
@zuzueeka
remove oracle from lido.py
f6b2646
@TheDZhon
fix: remove both oracle interfaces
3db4f90
@TheDZhon
fix: assign MANAGE_NODE_OPERATOR_ROLE in tests
e93adb5
@zuzueeka
Merge pull request #48 from lidofinance/fix/factories_for_stETH_topup
1a8d519 
Fix/factories for stETH topup
@rkolpakov
feat: modify top up factories to work with different stablecoins
a3e9ac3
@rkolpakov
feat: modify top up factories to work with different stablecoins
a7eafa7
@rkolpakov
chore: bump poetry lock brownie version to 1.19.3
0b1e3b3
@rkolpakov
chore: update readme
425f4a2
@rkolpakov
fix: make normalizeAmount safe for external callers
7e467c2
@rkolpakov
feat: pre deploy preparations
733ef7c
@TheDZhon
Merge branch 'master' into develop
62e1b1a
@TheDZhon
fix: add wf workaround
6ede292
@rkolpakov
fix: motion can be enacted several times in one period
e5b379c
@rkolpakov
feat: improve test creating top up motions several times
71d2944
@rkolpakov
Merge pull request #59 from lidofinance/fix/limits-checker-integratio…
2b93080 
…n-tests-motion-enactment

fix: motion can be enacted several times in one period
@rkolpakov
fix: missing comment parameter
feb440d
@rkolpakov
fix: pull fix from develop
480c3f1
@rkolpakov
fix: make allowed recipients factory functions external
9e4f813
@rkolpakov
fix: add zero validation top up evm factory
f4efe8e
@rkolpakov
fix: partially change memory params to calldata
3b718ad
@rkolpakov
fix: remove redundant asserts in builder
52b1b1d
@rkolpakov
fix: review fixes
35b01d1
@rkolpakov
fix: test fix
19e760e
@rkolpakov
fix: deploy fixes
bf6c9c5
@rkolpakov
test: bytecode verification
1c8f7c1
@bulbozaur
fix start fork
f6bf883
@zuzueeka
update lock files
25b774f
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
7 participants
@zuzueeka @arwer13 @TheDZhon @dgusakov @rkolpakov @bulbozaur @Psirex