Skip to content

Conversation

@xtbase
Copy link
Contributor

@xtbase xtbase commented Sep 5, 2025

What changed? Why?

  • support custom unmounted content instead of rendering a fixed styled button

Notes to reviewers

How has it been tested?

abcrane123 and others added 30 commits April 28, 2025 16:25
Signed-off-by: dependabot[bot] <[email protected]>
Co-authored-by: Daniel Schlabach <[email protected]>
Co-authored-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com>
Co-authored-by: Alissa Crane <[email protected]>
Co-authored-by: Copilot Autofix powered by AI <62310815+github-advanced-security[bot]@users.noreply.github.com>
Co-authored-by: Dan Cortes <[email protected]>
Co-authored-by: GarmashAlex <[email protected]>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
@vercel
Copy link

vercel bot commented Sep 5, 2025

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

Project Deployment Preview Comments Updated (UTC)
ock-minikit-example Ready Ready Preview Comment Sep 5, 2025 7:38am
onchainkit-playground Ready Ready Preview Comment Sep 5, 2025 7:38am
onchainkit-routes Ready Ready Preview Comment Sep 5, 2025 7:38am

Copy link

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull Request Overview

This PR adds support for custom unmounted content in the Wallet component instead of always rendering a fixed disabled "Connect Wallet" button during server-side rendering.

  • Adds an optional unmountedContent prop to the WalletProps interface
  • Updates the unmounted state rendering logic to use custom content when provided

Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.

@dgca dgca force-pushed the alpha branch 3 times, most recently from ca053ec to 108e128 Compare October 4, 2025 06:18
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Development

Successfully merging this pull request may close these issues.

4 participants