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

Feat: add WBTC deployment script and addresses for Sepolia and Mainnet #203

Open
wants to merge 7 commits into
base: development
Choose a base branch
from
Open

Feat: add WBTC deployment script and addresses for Sepolia and Mainnet #203

wants to merge 7 commits into from
+79 −52

Conversation

ricott1
Copy link
Contributor

@ricott1 ricott1 commented Sep 16, 2024 •
edited
Loading

What was the problem?

This PR resolves #202

How was it solved?

We added a new deployment script following the blueprint of USDT deployment.

How was it tested?

  • We deployed the contract on Lisk Sepolia and tested that the parameters were set succesfully.
  • We ran succesfully the deposit flow by interacting with the Sepolia L1 bridge and by transferring WBTC from L1 to L2.
  • We ran succesfully the withdrawal flow by interacting with the Sepolia L2 bridge and by withdrawing WBTC from L2 to L1.

@ricott1 ricott1 self-assigned this Sep 16, 2024
@ricott1 ricott1 requested a review from Incede September 16, 2024 13:48
@ricott1 ricott1 added the enhancement New feature or request label Sep 17, 2024
@ricott1 ricott1 marked this pull request as draft September 17, 2024 08:59
@ricott1
feat: add WBTC deployment script and addresses for mainnet and sepolia
222cb89 
Signed-off-by: ricott1 <ricott2@gmail.com>
@ricott1
Update WBTC address in testnet env file
ae6cdff 
Signed-off-by: ricott1 <ricott2@gmail.com>
@ricott1
Fix log typo
2caaf0a 
Signed-off-by: ricott1 <ricott2@gmail.com>
@ricott1 ricott1 force-pushed the feat/add-WBTC-deployment-script branch from 10bafa1 to 2caaf0a Compare September 17, 2024 14:19
@ricott1 ricott1 marked this pull request as ready for review September 19, 2024 13:52
@ricott1
Update default .env files (#205)
97089ae 
* Update L1_FORK_RPC_URL and L2_FORK_RPC_URL default values

Signed-off-by: ricott1 <ricott2@gmail.com>

* Add .env.devnet file and update README

Signed-off-by: ricott1 <ricott2@gmail.com>

* Update devnet env file and README; Improve comments in env files

Signed-off-by: ricott1 <ricott2@gmail.com>

* Make default URLs consistent

* Remove unused fork rpc URLs from public chain .env files

* Remove value of unused TEST_NETWORK_MNEMONIC variable

* Update the deployment description in the README

---------

Signed-off-by: ricott1 <ricott2@gmail.com>
@ricott1
feat: add WBTC deployment script and addresses for mainnet and sepolia
b779376 
Signed-off-by: ricott1 <ricott2@gmail.com>
@ricott1 ricott1 force-pushed the feat/add-WBTC-deployment-script branch from 12b46f5 to b779376 Compare September 19, 2024 14:06
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@Incede Incede Awaiting requested review from Incede

At least 2 approving reviews are required to merge this pull request.

Assignees

@ricott1 ricott1

Labels
enhancement New feature or request
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

Feat: Add WBTC deployment script
1 participant
@ricott1