Skip to content

fix: remove broken logo URL from Privy login modal#147

Merged
madeleine-c merged 1 commit intomainfrom
mcharity/fix-privy-login-logo
Feb 25, 2026
Merged

fix: remove broken logo URL from Privy login modal#147
madeleine-c merged 1 commit intomainfrom
mcharity/fix-privy-login-logo

Conversation

@madeleine-c
Copy link
Contributor

Summary

  • Remove the external logo URL (privy.io/privy-logo-dark.png) from the Privy provider appearance config, which was rendering as a broken image in the login modal
  • The logo now defaults to whatever is configured in the Privy dashboard

Test plan

  • Run npm run dev in privy-next-yield-demo/
  • Open http://localhost:3000 and trigger the login modal
  • Confirm the logo no longer shows a broken image

🤖 Generated with Claude Code

The external logo URL (privy.io/privy-logo-dark.png) was showing a
broken image in the login modal. Remove it so the logo defaults to
what's configured in the Privy dashboard.

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
Committed-By-Agent: claude
@vercel
Copy link

vercel bot commented Feb 25, 2026

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

Project Deployment Actions Updated (UTC)
privy-next-yield-demo Building Building Preview, Comment Feb 25, 2026 10:57pm
privy-react-farcaster Error Error Feb 25, 2026 10:57pm
privy-react-fiat-onramp Error Error Feb 25, 2026 10:57pm
privy-react-funding Error Error Feb 25, 2026 10:57pm
privy-react-permissionless Error Error Feb 25, 2026 10:57pm
privy-react-smart-wallets Error Error Feb 25, 2026 10:57pm
15 Skipped Deployments
Project Deployment Actions Updated (UTC)
privy-next-cross-app-connect Ignored Ignored Feb 25, 2026 10:57pm
privy-next-cross-app-provider Ignored Ignored Feb 25, 2026 10:57pm
privy-next-farcaster Ignored Ignored Feb 25, 2026 10:57pm
privy-next-farcaster-mini-app Ignored Ignored Feb 25, 2026 10:57pm
privy-next-fiat-onramp Ignored Ignored Feb 25, 2026 10:57pm
privy-next-funding Ignored Ignored Feb 25, 2026 10:57pm
privy-next-permissionless Ignored Ignored Feb 25, 2026 10:57pm
privy-next-session-keys Ignored Ignored Feb 25, 2026 10:57pm
privy-next-smart-wallets Ignored Ignored Feb 25, 2026 10:57pm
privy-next-solana Ignored Ignored Feb 25, 2026 10:57pm
privy-next-starter Ignored Ignored Feb 25, 2026 10:57pm
privy-next-wagmi Ignored Ignored Feb 25, 2026 10:57pm
privy-react-pwa Ignored Ignored Feb 25, 2026 10:57pm
privy-react-starter Ignored Ignored Feb 25, 2026 10:57pm
privy-react-whitelabel-starter Ignored Ignored Feb 25, 2026 10:57pm

Request Review

@madeleine-c madeleine-c merged commit 807187b into main Feb 25, 2026
22 of 27 checks passed
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.

2 participants