Skip to content

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 <bennett@umaproject.org>

* refactor

---------

Signed-off-by: bennett <bennett@umaproject.org>
Co-authored-by: nicholaspai <npai.nyc@gmail.com>
* 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 <bennett@umaproject.org>

* update deploy scripts

Signed-off-by: bennett <bennett@umaproject.org>

* conform to CCTPAdapterInterface

Signed-off-by: bennett <bennett@umaproject.org>

---------

Signed-off-by: bennett <bennett@umaproject.org>

* Update initial test

* Update zkSync SpokePool storage

* Extra test

Signed-off-by: Paul <108695806+pxrl@users.noreply.github.com>

* 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 <matthewcrice32@gmail.com>

* Clarify constructor arguments for zkSync

* readability in adapters

* Update deploy/059_deploy_lens_spokepool.ts

Co-authored-by: Matt Rice <matthewcrice32@gmail.com>

* 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 <bennett@umaproject.org>
Signed-off-by: Paul <108695806+pxrl@users.noreply.github.com>
Co-authored-by: bmzig <57361391+bmzig@users.noreply.github.com>
Co-authored-by: nicholaspai <npai.nyc@gmail.com>
Co-authored-by: nicholaspai <9457025+nicholaspai@users.noreply.github.com>
Co-authored-by: Matt Rice <matthewcrice32@gmail.com>
An approval is required to initiate a withdrawal of Circle bridged
(upgradable) USDC on Lens.
nicholaspai
nicholaspai previously approved these changes Apr 28, 2025
bmzig and others added 3 commits April 28, 2025 08:25
Signed-off-by: bennett <bennett@umaproject.org>
…USDC Through CCTP (#976)

Signed-off-by: bennett <bennett@umaproject.org>
@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 bmzig and nicholaspai May 7, 2025 20:47
@pxrl pxrl merged commit c441b3d into master Sep 9, 2025
10 checks passed
@pxrl pxrl deleted the pxrl/deployLens branch September 9, 2025 12:52
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.

4 participants