Skip to content

Fix wallet SDK dialog lifecycle - #33

Merged
coffee-the-dev merged 1 commit into
mainfrom
coffee/review-image
May 8, 2026
Merged

coffee-the-dev merged 1 commit into
mainfrom
coffee/review-image

Conversation

@coffee-the-dev

Copy link
Copy Markdown
Contributor

Summary

  • align the SDK iframe with the wallet-host-owned overlay/chrome model
  • send Porto-style drawer/floating resize hints from the SDK instead of resizing the iframe from host messages
  • close the SDK dialog after all pending requests resolve or when the host sends close
  • add a wallet-sdk patch changeset

Validation

  • pnpm --filter @abstract-foundation/wallet-sdk typecheck
  • pnpm --filter @abstract-foundation/wallet-sdk test

@vercel

vercel Bot commented May 8, 2026

Copy link
Copy Markdown

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

Project Deployment Actions Updated (UTC)
agw-nextjs Ready Ready Preview, Comment May 8, 2026 8:06pm
mpp-demo Ready Ready Preview, Comment May 8, 2026 8:06pm

Request Review

@chatgpt-codex-connector

Copy link
Copy Markdown

You have reached your Codex usage limits for code reviews. You can see your limits in the Codex usage dashboard.

@coffee-the-dev
coffee-the-dev merged commit 82b6ffb into main May 8, 2026
18 checks passed
@coffee-the-dev
coffee-the-dev deleted the coffee/review-image branch May 8, 2026 20:07
@abstract-release-bot abstract-release-bot Bot mentioned this pull request May 8, 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.

1 participant