Skip to content

feat(protocol-contracts): add depositWithPermit function in OperatorStaking#1564

Merged
melanciani merged 4 commits intomainfrom
melanciani/716/protocol-contracts/staking/add_deposit_permit_function
Dec 12, 2025
Merged

feat(protocol-contracts): add depositWithPermit function in OperatorStaking#1564
melanciani merged 4 commits intomainfrom
melanciani/716/protocol-contracts/staking/add_deposit_permit_function

Conversation

@melanciani
Copy link
Copy Markdown
Contributor

@melanciani melanciani commented Dec 12, 2025

to benefit from the token's ERC 2612 and allow approval + deposit in a single tx for EOA accounts

refs https://github.com/zama-ai/protocol-apps-internal/issues/22
closes https://github.com/zama-ai/fhevm-internal/issues/716

@melanciani melanciani requested review from arr00 and jatZama December 12, 2025 14:53
@melanciani melanciani requested a review from a team as a code owner December 12, 2025 14:53
@cla-bot cla-bot Bot added the cla-signed label Dec 12, 2025
@mergify
Copy link
Copy Markdown

mergify Bot commented Dec 12, 2025

🧪 CI Insights

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

🟢 All jobs passed!

But CI Insights is watching 👀

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.

Looks good for the most part, just some small comments.

Comment thread protocol-contracts/staking/test/OperatorStaking.test.ts Outdated
Comment thread protocol-contracts/staking/test/OperatorStaking.test.ts Outdated
@melanciani melanciani requested a review from arr00 December 12, 2025 16:13
@melanciani melanciani mentioned this pull request Dec 12, 2025
jatZama
jatZama previously approved these changes Dec 12, 2025
@melanciani melanciani merged commit 371603f into main Dec 12, 2025
64 checks passed
@melanciani melanciani deleted the melanciani/716/protocol-contracts/staking/add_deposit_permit_function branch December 12, 2025 17: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