Skip to content

Feat(SystemSettings): SIP-193 Reduce size for SystemSettings#1626

Closed
leckylao wants to merge 94 commits intodevelopfrom
feat/fix-system-settings-size
Closed

Feat(SystemSettings): SIP-193 Reduce size for SystemSettings#1626
leckylao wants to merge 94 commits intodevelopfrom
feat/fix-system-settings-size

Conversation

@leckylao
Copy link
Copy Markdown
Contributor

@leckylao leckylao commented Dec 8, 2021

No description provided.

- created libraries folder
- moving SafeDecimalMath into libraries
- adding decimals into SafeDecimalMath
- Adding require check for price;
- Adding setup for test;
- Adding more rounds of price feeds for test;
- Adding dynamic fee in systemSettings in test
- removed fastForward as startTime and Now is not equal
- Rename priceWeight to RoundDecay
- adding cap 100% for max exchangeFeeRate
- Added return for invalid rate
- linking libraries in setup
- disable dynamic fee when price feeds less than 2 rounds
- disable dynamic fee on exchanger spec settle
- Adding more rounds for rates
- Rollback as no cache for _getRateAndUpdatedTime
- Disable Dynamic fee on test with base exchange fee rate
- Adding more rounds on rates;
- Disable Dynamic fee in order for debt calculation
- Adding generateSolidity into configure-system-settings
- Adding MAX_DYNAMIC_FEE default to 1
- remove comment with system value
- remove nonreentrant
- removed swing trade double fee
- created getExchangeDynamicFeeData for combine external call
- _setRate skip writing if same rate
…mic fee

- remove duplicated code in _calculateFeeRateFromExchangeSynths
- Added test cases for up/down price differential
- Disable dynamic fee to neglect it in systemSettings
- change ConfigureSystemSettings to only change if zero
@leckylao
Copy link
Copy Markdown
Contributor Author

leckylao commented Dec 8, 2021

Wrong init branch

@leckylao leckylao closed this Dec 8, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant