Skip to content

Comments

chore: deploy HyperliquidDepositHandler after audit of Jan 19#1313

Open
grasphoper wants to merge 2 commits intomasterfrom
if/hldeposithandler-afterauditjan19
Open

chore: deploy HyperliquidDepositHandler after audit of Jan 19#1313
grasphoper wants to merge 2 commits intomasterfrom
if/hldeposithandler-afterauditjan19

Conversation

@grasphoper
Copy link
Contributor

@grasphoper grasphoper commented Feb 13, 2026

Also fixes HyperliquidDepositHandler's deployment script

Closes ACP-53

Signed-off-by: Ihor Farion <ihor@umaproject.org>
@linear
Copy link

linear bot commented Feb 13, 2026

Signed-off-by: Ihor Farion <ihor@umaproject.org>
Copy link
Contributor

@tbwebb22 tbwebb22 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

looks like owner and signer are both currently the dev wallet - is this what we want?

@grasphoper
Copy link
Contributor Author

@tbwebb22 yea that's fine for now. Once backend team is done with testing with these contracts, we can change signer to prod backend EOA first, then transfer ownership to a multisig; that's the plan

@grasphoper
Copy link
Contributor Author

@tbwebb22 yea that's fine for now. Once backend team is done with testing with these contracts, we can change signer to prod backend EOA first, then transfer ownership to a multisig; that's the plan

When they do it and we change the singer / owner we can merge this, but for now can just stay as a PR

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.

2 participants