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

PP-885/explore-solution-b2 #260

Merged

antomor merged 14 commits into main from PP-885/explore-solution-b2

Mar 19, 2024

Collaborator

franciscotobar commented Jan 29, 2024 •

edited

Loading

What

Implements solution B2

Why

Necessary to execute a contract while deploying a smart wallet

Refs

franciscotobar changed the title ~~refactor: first attempt~~ PP-885/explore-solution-b2

franciscotobar requested a review from antomor

January 29, 2024 03:23


          refactor: first attempt

a813688

franciscotobar force-pushed the PP-885/explore-solution-b2 branch from 71a78b3 to a813688 Compare

January 29, 2024 03:39

franciscotobar added 2 commits

February 1, 2024 00:32


          refactor: integration test

bf8e85b


          refactor: boltz components

0c8610a

antomor requested changes

View reviewed changes

contracts/BoltzDeployVerifier.sol Outdated Show resolved Hide resolved

package.json Show resolved Hide resolved

package.json Outdated Show resolved Hide resolved

scripts/GasEstimation.ts Outdated Show resolved Hide resolved

scripts/GasEstimation.ts Outdated Show resolved Hide resolved

test/RelayHub.test.ts Outdated Show resolved Hide resolved

.husky/pre-push Show resolved Hide resolved

antomor and others added 7 commits

February 6, 2024 20:10


          chore: apply small changes to the tests

7e71cbe


          fix: change the estimation script to include the full boltz template

b62a2af


          fix: cover the minimal boltz template with the gas estimation

2cb5215


          fix: remove hard-coded gas and add some checks on deployment

6a5e141


          refactor: minimal boltz

865c643


          refactor: destination contract

93eb638


          test: fix

5d9f414

antomor reviewed

View reviewed changes

test/relayclient/RelayClient.test.ts Outdated Show resolved Hide resolved

test/utils/TestUtils.ts Outdated Show resolved Hide resolved

antomor reviewed

View reviewed changes

test/relayserver/RelayServer.test.ts Outdated Show resolved Hide resolved

antomor reviewed

View reviewed changes

test/relayserver/RelayServer.test.ts Outdated Show resolved Hide resolved

antomor reviewed

View reviewed changes

test/smartwallet/baseSmartWallet.test.ts Outdated Show resolved Hide resolved

antomor reviewed

View reviewed changes

test/smartwallet/baseSmartWallet.test.ts Outdated Show resolved Hide resolved

antomor reviewed

View reviewed changes

test/RelayHub.test.ts Outdated Show resolved Hide resolved

antomor reviewed

View reviewed changes

test/relayserver/RelayServer.test.ts Outdated Show resolved Hide resolved


          refactor: pr comments

13da238

franciscotobar requested a review from antomor

February 16, 2024 20:03

antomor reviewed

View reviewed changes

test/RelayHub.test.ts Outdated Show resolved Hide resolved

antomor reviewed

View reviewed changes

test/utils/TestUtils.ts Outdated Show resolved Hide resolved

franciscotobar and others added 2 commits

February 21, 2024 07:38


          refactor: pr comments

02bf404


          chore: updated dependencies

109f032

antomor marked this pull request as ready for review

February 28, 2024 17:41


          test: add some tests to show the interaction with the server without …

5573c73

…client

antomor approved these changes

View reviewed changes

Collaborator

antomor left a comment

LGTM

antomor merged commit 1c0c81f into main

1 of 2 checks passed

antomor deleted the PP-885/explore-solution-b2 branch

March 19, 2024 15:00

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet