[SIP5.9] Upgrading AdmiralsQuarters contracts on all chains to support Merkl claiming

Governance Summer Improvement Proposals [SIPs]
, , ,
1

Following the → [RFC] Transfer Vault Rewards from staking to Merkl and enable tokenisation of Vault Shares - #15 by rspa_StableLab

Summary

We’ve recently deployed new versions of the Admirals Quarters contract on all supported chains. Admirals Quarters (AQ) is a non-protocol contract that allows the Summer.fi team to offer better UX. Similar to Morpho’s Bundler contract. There is one method on this contract that does require a role to be granted by the central Protocol Access Manager contract - see below.

Motivation

The new contracts require the ADMIRALS_QUARTERS role be granted to it - it’s required by the unstakeAndWithdrawAssets method. Any further updates to AdmiralQuarters contract will not require this role - ad staking module gets deprecated.
The new AQ allows users to bundle Merkl claiming operation with other operations (e.g. withdraw, deposit etc).

Specifications

Newly deployed AdmiralsQuarters contracts:

- base: 0x4e92071F9BC94011419Dc03fEaCA32D11241313a

- arbitrum: 0x1db04f01386c6BE2d22b7947236d8ACc05901219

- mainnet: 0xD03bD9Ef8c72Aee3DBb5b8DF83c479D847622Dba

- sonic: 0xa514a99b3584D152b2BE9cBe3e7B34Ad40954410

# Specifications

### Actions

1. On base:

  • Grant ADMIRALS_QUARTERS_ROLE to AdmiralsQuarters at 0x4e92071F9BC94011419Dc03fEaCA32D11241313a

2a. Send cross-chain proposal to arbitrum to:

  • Grant ADMIRALS_QUARTERS_ROLE to AdmiralsQuarters at 0x1db04f01386c6BE2d22b7947236d8ACc05901219

2b. Send cross-chain proposal to mainnet to:

  • Grant ADMIRALS_QUARTERS_ROLE to AdmiralsQuarters at 0xD03bD9Ef8c72Aee3DBb5b8DF83c479D847622Dba

2c. Send cross-chain proposal to sonic to:

  • Grant ADMIRALS_QUARTERS_ROLE to AdmiralsQuarters at 0xa514a99b3584D152b2BE9cBe3e7B34Ad40954410

@Recognized_Delegates Tally | Lazy Summer DAO (Official) | SIP5.9: Multi-Chain AdmiralsQuarters Role Setup

3 Likes
2

I am in support of this one! Thank you @halaprix for bringing it up, as it is one of several backend cleanups that will bring smoother SUMR interactions going forward.

3

In light of requirements of the transferability readiness, this is just what the doctor ordered. Very quick turnaround too. Supported.

2 Likes