Skip to content

Fix/erc20 bridgeable invalid sender - #399

Closed
Jayy4rl wants to merge 13 commits into
Perfect-Abstractions:mainfrom
Jayy4rl:fix/erc20-bridgeable-invalid-sender
Closed

Jayy4rl wants to merge 13 commits into
Perfect-Abstractions:mainfrom
Jayy4rl:fix/erc20-bridgeable-invalid-sender

Conversation

@Jayy4rl

@Jayy4rl Jayy4rl commented Sep 16, 2026

Copy link
Copy Markdown
Contributor

Summary

Bridgeable mod and facet reverts ERC20InvalidReceiver for this check

 if (_from == address(0)) {
            revert ERC20InvalidReceiver(address(0));
        }

Changes Made

Edited the revert in both facet and mod to revert ERC20InvalidSender. The error cascaded into the tests, so i edited that too in the unit tests and the trees.

Closes #398

@changeset-bot

changeset-bot Bot commented Sep 16, 2026

Copy link
Copy Markdown

⚠️ No Changeset found

Latest commit: 90856a4

Merging this PR will not cause a version bump for any packages. If these changes should not result in a new version, you're good to go. If these changes should result in a version bump, you need to add a changeset.

This PR includes no changesets

When changesets are added to this PR, you'll see the packages that this PR includes changesets for and the associated semver types

Click here to learn what changesets are, and how to add one.

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

@netlify

netlify Bot commented Sep 16, 2026

Copy link
Copy Markdown

👷 Deploy request for compose-diamonds pending review.

Visit the deploys page to approve it

Name Link
🔨 Latest commit 90856a4

@Jayy4rl Jayy4rl closed this Sep 16, 2026
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.

[BUG] Crosschain Burn Error Selector

2 participants