Skip to content

feat: widget context cleanup#2365

Merged
chaitanyapotti merged 2 commits intomasterfrom
feat/widget-context-cleanup
Feb 5, 2026
Merged

feat: widget context cleanup#2365
chaitanyapotti merged 2 commits intomasterfrom
feat/widget-context-cleanup

Conversation

@lionellbriones
Copy link
Copy Markdown
Contributor

@lionellbriones lionellbriones commented Feb 5, 2026

Motivation and Context

Jira Link:

Description

Remove extra props for keys which can be found in uiConfig already. For example developers gets the chainnamespaces from uiConfig and sometimes gets from useWidget directly but it's actually the same

How has this been tested?

Screenshots (if appropriate):

Types of changes

  • Bug fix (non-breaking change which fixes an issue)
  • New feature (non-breaking change which adds functionality)
  • Breaking change (fix or feature that would cause existing functionality to not work as expected)

Checklist:

  • My code follows the code style of this project. (run lint)
  • My change requires a change to the documentation.
  • I have updated the documentation accordingly.
  • I have added tests to cover my changes.
  • All new and existing tests passed.
  • My code requires a db migration.

Note

Low Risk
Mostly a context/API cleanup that reroutes reads through uiConfig; risk is limited to potential undefined config fields (e.g., walletRegistry) causing runtime errors in affected screens/analytics.

Overview
Reduces WidgetContext surface area by removing duplicated fields (appName, chainNamespaces, walletRegistry, initialAuthenticationMode) and deriving values from uiConfig instead (including computing appLogo from logoDark/logoLight).

Updates modal UI containers (Root, ConnectWallet, ConnectWalletChainFilter, Widget) to read chainNamespaces, walletRegistry, and initialAuthenticationMode from uiConfig, and refactors Widget to reuse a single Root element for both modal and embed rendering.

Cleans up LoginModal by no longer storing chainNamespaces/walletRegistry separately, using the isDark getter directly when mounting, and updating open-event analytics to use values from uiConfig.

Written by Cursor Bugbot for commit e75a7da. This will update automatically on new commits. Configure here.

@vercel
Copy link
Copy Markdown

vercel Bot commented Feb 5, 2026

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

Project Deployment Actions Updated (UTC)
web3auth-web Ready Ready Preview, Comment Feb 5, 2026 8:20am

Request Review

@chaitanyapotti chaitanyapotti merged commit a30b792 into master Feb 5, 2026
7 checks passed
@chaitanyapotti chaitanyapotti deleted the feat/widget-context-cleanup branch February 5, 2026 09:09
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