Skip to content

feat(protocol-contracts): be able to set a claimer for claimRewards in OperatorStaking#1549

Merged
melanciani merged 3 commits intomainfrom
melanciani/716/protocol-contracts/staking/gate_claim_rewards_function
Dec 12, 2025
Merged

feat(protocol-contracts): be able to set a claimer for claimRewards in OperatorStaking#1549
melanciani merged 3 commits intomainfrom
melanciani/716/protocol-contracts/staking/gate_claim_rewards_function

Conversation

@melanciani
Copy link
Copy Markdown
Contributor

@melanciani melanciani commented Dec 11, 2025

@melanciani melanciani requested review from arr00 and jatZama December 11, 2025 11:37
@cla-bot cla-bot Bot added the cla-signed label Dec 11, 2025
@melanciani melanciani changed the title feat(protocol-contracts): be able to set a claimer for claimRewards i… feat(protocol-contracts): be able to set a claimer for claimRewards in OperatorStaking Dec 11, 2025
@mergify
Copy link
Copy Markdown

mergify Bot commented Dec 11, 2025

🧪 CI Insights

Here's what we observed from your CI run for cce18c5.

🟢 All jobs passed!

But CI Insights is watching 👀

@melanciani melanciani force-pushed the melanciani/716/protocol-contracts/staking/gate_claim_rewards_function branch from 1c320d8 to bf3ea13 Compare December 11, 2025 12:26
@melanciani melanciani marked this pull request as ready for review December 11, 2025 12:26
@melanciani melanciani requested a review from a team as a code owner December 11, 2025 12:26
jatZama
jatZama previously approved these changes Dec 12, 2025
Copy link
Copy Markdown
Member

@jatZama jatZama left a comment

Choose a reason for hiding this comment

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

Good job 👍
Maybe ask @arr00 to double check this is what he requested, but it looks fine for me.

Comment thread protocol-contracts/staking/contracts/OperatorRewarder.sol Outdated
Comment thread protocol-contracts/staking/test/OperatorRewarder.test.ts Outdated
Copy link
Copy Markdown
Collaborator

@arr00 arr00 left a comment

Choose a reason for hiding this comment

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

Great approach. Thanks for this!

Comment thread protocol-contracts/staking/contracts/OperatorRewarder.sol Outdated
@melanciani melanciani merged commit 2bf98c8 into main Dec 12, 2025
64 checks passed
@melanciani melanciani deleted the melanciani/716/protocol-contracts/staking/gate_claim_rewards_function branch December 12, 2025 10:05
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants