Skip to content

chore: Deploy Lens SpokePool & Adapter #978

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

Open
wants to merge 21 commits into
base: master
Choose a base branch
from
Open

chore: Deploy Lens SpokePool & Adapter #978

wants to merge 21 commits into from

Conversation

pxrl
Copy link
Contributor

@pxrl pxrl commented Apr 28, 2025

No description provided.

nicholaspai and others added 6 commits April 8, 2025 11:14
* feat: Deploy Linea Adapter and SpokePool with CCTP V2 support

- TODO: Deploy SpokePool

* Add LineaSpokePool deployment
* improve: query shared bridge address dynamically

Signed-off-by: bennett <[email protected]>

* refactor

---------

Signed-off-by: bennett <[email protected]>
Co-authored-by: nicholaspai <[email protected]>
* feat: Support Lens Circle bridged USDC in SpokePool

* immutable

* Add zkSync contract gap

* Centralise defs

* Migrate to zkSync SpokePool

* Update

* update

* Update contracts/ZkSync_SpokePool.sol

* revert unnecessary change

* Update comments

* Fix deployment script

* Simplify

* Update comments

* Update import

* fixes

* improve: support upgradeable USDC in zkstack adapters (#940)

* improve: support upgradeable USDC in zkstack adapters

Signed-off-by: bennett <[email protected]>

* update deploy scripts

Signed-off-by: bennett <[email protected]>

* conform to CCTPAdapterInterface

Signed-off-by: bennett <[email protected]>

---------

Signed-off-by: bennett <[email protected]>

* Update initial test

* Update zkSync SpokePool storage

* Extra test

Signed-off-by: Paul <[email protected]>

* Update test/evm/hardhat/chain-specific-spokepools/ZkSync_SpokePool.ts

* Apply suggestions from code review

* Harden constructor

* Expand & fix

* fix

* Apply suggestions from code review

* Add constructor checks to Adapters, update deploy script

* Update 059_deploy_lens_spokepool.ts

* Update contracts/ZkSync_SpokePool.sol

Co-authored-by: Matt Rice <[email protected]>

* Clarify constructor arguments for zkSync

* readability in adapters

* Update deploy/059_deploy_lens_spokepool.ts

Co-authored-by: Matt Rice <[email protected]>

* Move asserts

* Remove if (address(_l2Usdc) != zero)

* Fix deploy assert

* Revert "Remove if (address(_l2Usdc) != zero)"

This reverts commit ffeaade.

* Rename constructor param

* Remove SHARED_BRIDGE

---------

Signed-off-by: bennett <[email protected]>
Signed-off-by: Paul <[email protected]>
Co-authored-by: bmzig <[email protected]>
Co-authored-by: nicholaspai <[email protected]>
Co-authored-by: nicholaspai <[email protected]>
Co-authored-by: Matt Rice <[email protected]>
An approval is required to initiate a withdrawal of Circle bridged
(upgradable) USDC on Lens.
nicholaspai
nicholaspai previously approved these changes Apr 28, 2025
@pxrl pxrl force-pushed the pxrl/deployLens branch from e6ff80c to a84bc4a Compare April 28, 2025 13:56
@pxrl pxrl dismissed nicholaspai’s stale review April 28, 2025 13:57

I force-pushed after approval.

@pxrl pxrl requested a review from nicholaspai April 28, 2025 13:57
bmzig
bmzig previously approved these changes Apr 28, 2025
@pxrl pxrl added the do not merge do not merge label Apr 28, 2025
@pxrl pxrl dismissed bmzig’s stale review April 28, 2025 19:14

Adapter must be redployed.

@pxrl pxrl changed the base branch from audit-zkstack-usdc to pxrl/fixApproval April 29, 2025 08:56
bmzig
bmzig previously approved these changes Apr 29, 2025
pxrl added 2 commits April 30, 2025 11:57
This variable is no longer used. Removing it addresses a warning being
emitted by solc.
Base automatically changed from pxrl/fixApproval to audit-zkstack-usdc April 30, 2025 14:36
nicholaspai
nicholaspai previously approved these changes Apr 30, 2025
@pxrl pxrl removed the do not merge do not merge label May 2, 2025
Base automatically changed from audit-zkstack-usdc to master May 2, 2025 16:24
@pxrl pxrl dismissed stale reviews from nicholaspai and bmzig May 2, 2025 16:24

The base branch was changed.

@pxrl pxrl requested review from nicholaspai and bmzig May 7, 2025 20:47
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.

3 participants