-
Notifications
You must be signed in to change notification settings - Fork 16
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
🚀 Andromeda #89
🚀 Andromeda #89
Conversation
* add missing parameter * lint * fix
source = "mintable-token" | ||
targetPreset = "<%= settings.target_preset %>" | ||
options.salt = "<%= settings.salt %>" | ||
source = "mintable-token:1.3" |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Need to use 1.7 and decimals 6
source = "mintable-token:1.3" | |
source = "mintable-token:1.7" | |
options.decimals = "6" |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This was deployed as is though? I don't think we should change it if that's the case.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
We need to have fUSDC to have same decimals as real usdc because all the tests and potentially integrators implementations assume 18 decimals. Which will be a problem when we deploy to mainnet. We need to have a testnet deploy that is close to mainnet to avoid such issues
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Need to make sure we can withdraw collateral (debt burn issue),
WIP PR #92
* Add mining, debt burning, and sUSD swapping in Ethereum end-to-end testing New functionalities are implemented to enrich the capabilities of Ethereum end-to-end testing. These include the addition of block mining, debt burning, and swapping to sUSD. The changes also extend the tasks in the Liquidity Provider e2e test, including elements such as account creation and sUSD deposit, thereby boosting the robustness and coverage of the testing script. * Add functionality to undelegate collateral
* base mainnet andromeda setup * pretty * limit to $50k LP and $10k OI as updated SIP-348 * added some todos * mainnet changes * fix url * fix include order * fixes * fixes * build success * deniers * susdc override * fix * remove susdc from deduction * update parameters * update settlement delay * fixes --------- Co-authored-by: cavalier_eth <91091124+cavalier-eth@users.noreply.github.com> Co-authored-by: Noah Litvin <335975+noahlitvin@users.noreply.github.com> Co-authored-by: kaleb <kalebkeny@gmail.com>
cc6340c
to
f52f07f
Compare
defaultValue = "0.0005" | ||
|
||
[setting.perpsEthEndorsedLiquidator] | ||
defaultValue = "0x11233749514Ab8d00C0A5873DF7428b3db70030f" |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This value is the same as testnet. If it should be - nevermind
defaultValue = "0.0005" | ||
|
||
[setting.perpsBtcEndorsedLiquidator] | ||
defaultValue = "0x11233749514Ab8d00C0A5873DF7428b3db70030f" |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Same comment here too: this value is the same as testnet. If it should be - nevermind then
2818f4c
to
2132c6d
Compare
…b.com/Synthetixio/synthetix-deployments into add-referrer-and-other-items-to-perps
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I did check everything and made a few fixes. There are couple of copypasted addresses to double-check and we still have a few TODOs.
Existing tests are green (NOTE: we don't have tests for mainnet deployment yet!!!)
No description provided.