Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

fix: pass though logo and name to cb wallet #1808

Merged
merged 1 commit into from
Jan 13, 2025

Conversation

alessey
Copy link
Contributor

@alessey alessey commented Jan 13, 2025

What changed? Why?
Pass through appLogo and appName to cb smart wallet from wallet modal

fixes: #1789

before after
image image

Notes to reviewers

How has it been tested?

@alessey alessey force-pushed the alessey/fix-wallet-modal-appearance branch from 8505fc1 to 8312da3 Compare January 13, 2025 20:31
Copy link

vercel bot commented Jan 13, 2025

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Comments Updated (UTC)
onchainkit-coverage ✅ Ready (Inspect) Visit Preview 💬 Add feedback Jan 13, 2025 8:34pm
onchainkit-playground ✅ Ready (Inspect) Visit Preview 💬 Add feedback Jan 13, 2025 8:34pm
onchainkit-routes ✅ Ready (Inspect) Visit Preview 💬 Add feedback Jan 13, 2025 8:34pm

@alessey alessey merged commit a0d69b6 into main Jan 13, 2025
16 checks passed
@alessey alessey deleted the alessey/fix-wallet-modal-appearance branch January 13, 2025 20:59
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Development

Successfully merging this pull request may close these issues.

Bug: Wallet Modal breaks app appearance
3 participants