Skip to content

feat(explorer): Gated Send Tx for Foreclosed Applications#466

Merged
brunomenezes merged 1 commit into
mainfrom
feat/disallow-send-tx-to-foreclosed-apps
Jun 22, 2026
Merged

feat(explorer): Gated Send Tx for Foreclosed Applications#466
brunomenezes merged 1 commit into
mainfrom
feat/disallow-send-tx-to-foreclosed-apps

Conversation

@brunomenezes

Copy link
Copy Markdown
Collaborator

Summary

This PR implements a safeguard that prevents users from accidentally trying to send transactions to foreclosed applications. It blocks and avoids the experience of getting a revert upon sending the transaction and provides clear, contextual feedback on why the action is restricted.

User Experience (UX) & Accessibility Changes

  • The "Send" Button: Automatically disables if the application is "Foreclosed."
  • Contextual Tooltip: Clicking the disabled button triggers a styled, multiline tooltip reading: “The application is foreclosed. You can no longer send transactions.”
  • Accessibility (a11y): Added aria-disabled attributes to ensure screen readers properly communicate the button's locked state.

Risk & Verification

  • Edge Cases Handled: Standard flows (wallet connection checks, network switching modals) still operate for active apps.
  • Automated Coverage: Added 8 test cases covering all transaction types (Ether, ERC-20, ERC-721, ERC-1155) to guarantee no regressions in the core send-tx workflow.

Extra details

Click the GIF below to see it in a bigger resolution.

Trying to Send a Tx to a Foreclosed Application 2026-06-22 18 49 23

…eclosed.

* Add a set of tests for the affected component SendMenu.
@brunomenezes brunomenezes added this to the 2.0.0 milestone Jun 22, 2026
@brunomenezes brunomenezes self-assigned this Jun 22, 2026
@vercel

vercel Bot commented Jun 22, 2026

Copy link
Copy Markdown

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Actions Updated (UTC)
dave Ready Ready Preview, Comment Jun 22, 2026 6:01pm
rollups-explorer-arbitrum-mainnet Ready Ready Preview, Comment Jun 22, 2026 6:01pm
rollups-explorer-arbitrum-sepolia Ready Ready Preview, Comment Jun 22, 2026 6:01pm
rollups-explorer-base-mainnet Ready Ready Preview, Comment Jun 22, 2026 6:01pm
rollups-explorer-base-sepolia Ready Ready Preview, Comment Jun 22, 2026 6:01pm
rollups-explorer-mainnet Ready Ready Preview, Comment Jun 22, 2026 6:01pm
rollups-explorer-optimism-mainnet Ready Ready Preview, Comment Jun 22, 2026 6:01pm
rollups-explorer-optimism-sepolia Ready Ready Preview, Comment Jun 22, 2026 6:01pm
rollups-explorer-sepolia Ready Ready Preview, Comment Jun 22, 2026 6:01pm
rollups-explorer-workshop Ready Ready Preview, Comment Jun 22, 2026 6:01pm

Request Review

@brunomenezes brunomenezes moved this from Todo to Waiting review in Rollups SDK Jun 22, 2026
@github-project-automation github-project-automation Bot moved this from Waiting review to Waiting Merge in Rollups SDK Jun 22, 2026
@brunomenezes brunomenezes merged commit 93c7b85 into main Jun 22, 2026
20 of 21 checks passed
@brunomenezes brunomenezes deleted the feat/disallow-send-tx-to-foreclosed-apps branch June 22, 2026 19:22
@github-project-automation github-project-automation Bot moved this from Waiting Merge to Done in Rollups SDK Jun 22, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

Status: Done

Development

Successfully merging this pull request may close these issues.

2 participants