-
Notifications
You must be signed in to change notification settings - Fork 37
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
New deployment of audited contracts #501
Conversation
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.
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.
New deployment, new day!
@@ -1,6 +1,6 @@ | |||
{ | |||
"name": "@gnosis.pm/dex-contracts", | |||
"version": "0.1.4", | |||
"version": "0.2.0", |
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.
🚀 🎉 💥 🤘 🎸
@@ -66,7 +66,7 @@ | |||
"devDependencies": { | |||
"@gnosis.pm/mock-contract": "^3.0.8", | |||
"@gnosis.pm/owl-token": "^3.1.0", | |||
"@gnosis.pm/solidity-data-structures": "^1.2.2", | |||
"@gnosis.pm/solidity-data-structures": "^1.2.4", |
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.
Just curious what this version bump is for?
Missed it in the PR description.
This is the long-awaited deployment of the audited contracts.
It bumps the version of the "@gnosis.pm/solidity-data-structures", as I used the new library for the deployments.
Note: Actually, I was running the deployment with the old migration scripts, as the current migration scripts still need to be fixed
I also decided for version 0.2.0, as we have the new contracts, but I did not want to go for 1.0.0, as the migration scripts are still to be fixed.
I was also listing the stablecoins and placed some basic orders on the exchange.
testplan:
Check that MAX_TOUCHED_ORDERS ==30 in etherscan
https://etherscan.io/address/0x6F400810b62df8E13fded51bE75fF5393eaa841F#code
Check that new library was used:
https://etherscan.io/address/0xED4d05496C71e71cC2A8726af1242C22108d1761#code
and address(0x) can not be added.
PS:
This PR replaces #498