Skip to content

Increase vesting wallet native erc20s comment prefix#6398

Closed
gonzaotc wants to merge 20 commits intoOpenZeppelin:masterfrom
gonzaotc:increase-vesting-wallet-native-erc20s-comment-prefix
Closed

Increase vesting wallet native erc20s comment prefix#6398
gonzaotc wants to merge 20 commits intoOpenZeppelin:masterfrom
gonzaotc:increase-vesting-wallet-native-erc20s-comment-prefix

Conversation

@gonzaotc
Copy link
Contributor

@gonzaotc gonzaotc commented Mar 9, 2026

Patches L-01

Increases the VestingWallet comment about native erc20 behavior prefix from NOTE to WARNING

PR Checklist

  • Tests
  • Documentation
  • Changeset entry (run npx changeset add)

@gonzaotc gonzaotc requested a review from a team as a code owner March 9, 2026 14:22
@changeset-bot
Copy link

changeset-bot bot commented Mar 9, 2026

🦋 Changeset detected

Latest commit: 977bcbe

The changes in this PR will be included in the next version bump.

This PR includes changesets to release 1 package
Name Type
openzeppelin-solidity Minor

Not sure what this means? Click here to learn what changesets are.

Click here if you're a maintainer who wants to add another changeset to this PR

@gonzaotc gonzaotc closed this Mar 9, 2026
@coderabbitai
Copy link
Contributor

coderabbitai bot commented Mar 9, 2026

Caution

Review failed

The pull request is closed.

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: Repository UI

Review profile: CHILL

Plan: Pro

Run ID: 4d14bfa3-7172-441d-9fc3-a7ff68ab241d

📥 Commits

Reviewing files that changed from the base of the PR and between 45f032d and 977bcbe.

📒 Files selected for processing (7)
  • .changeset/metal-chicken-dress.md
  • contracts/finance/VestingWallet.sol
  • contracts/interfaces/draft-IERC3009.sol
  • contracts/mocks/token/ERC20TransferAuthorizationBlockNumberMock.sol
  • contracts/token/ERC20/extensions/ERC20TransferAuthorization.sol
  • test/helpers/eip712-types.js
  • test/token/ERC20/extensions/ERC20TransferAuthorization.test.js

Walkthrough

This PR introduces ERC-3009 transfer with authorization support to the package. A new ERC20TransferAuthorization contract implements gas-less transfers via EIP-712 signatures with keyed sequential nonces and time-window validation. Supporting additions include new ERC-3009 interfaces, a block-number-based mock variant for testing, test helpers with EIP-712 type definitions, and a comprehensive test suite. A documentation update in VestingWallet.sol changes a NOTE descriptor to WARNING. A changelog entry documents the minor version bump.

Suggested labels

documentation

✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Post copyable unit tests in a comment

Tip

Try Coding Plans. Let us write the prompt for your AI agent so you can ship faster (with fewer bugs).
Share your feedback on Discord.


Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands and usage tips.

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.

3 participants