fix(ui): Display admin-created Web3 wallet in UserProfile - #7981

Merged
wobsoriano merged 2 commits into
mainfrom
daniel/user-3812-wallet-not-showing-up-in-userprofile
Apr 23, 2026
Merged

fix(ui): Display admin-created Web3 wallet in UserProfile#7981
wobsoriano merged 2 commits into
mainfrom
daniel/user-3812-wallet-not-showing-up-in-userprofile

Conversation

@dmoerner

@dmoernerdmoerner commented Mar 4, 2026

Copy link
Copy Markdown
Contributor

Description

When a user is created with a web3wallet using BAPI, the verification strategy is admin instead of the actual provider for the web3 Wallet. This currently prevents the web3 Wallet from displaying in the User Profile, since we do not know the name or logo. (And we will not know it until the user re-verifies or signs in with the web3 wallet.) Add a fallback display for such web3wallets which just shows the shortened wallet signature. This is coupled with a backend change which will update the strategy on successful sign in, so that after a sign in with the web3 wallet the correct logo and name will be displayed.

Part of USER-3812

Screenshot 2026-03-04 at 16-22-42 clerk-js Sandbox

Checklist

  • pnpm test runs as expected.
  • pnpm build runs as expected.

Type of change

  • 🐛 Bug fix
  • 🌟 New feature
  • 🔨 Breaking change
  • 📖 Refactoring / dependency upgrade / documentation
  • other:

Summary by CodeRabbit

  • New Features

    • Improved web3 wallet rendering in user profiles, now displaying provider names (e.g., MetaMask) alongside wallet addresses when provider data is available.
  • Tests

    • Added comprehensive test coverage for web3 wallet display scenarios, including known providers, administrator-added wallets, and unknown wallet types.

When a user is created with a web3wallet using BAPI, the verification
strategy is `admin` instead of the actual provider for the web3 Wallet.
This currently prevents the web3 Wallet from displaying in the User
Profile, since we do not know the name or logo. (And we will not know it
until the user re-verifies or signs in with the web3 wallet.) Add a
fallback display for such web3wallets which just shows the shortened
wallet signature.
@vercel

vercelBot commented Mar 4, 2026

Copy link
Copy Markdown

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

ProjectDeploymentActionsUpdated (UTC)
clerk-js-sandboxReadyReadyPreview, CommentApr 23, 2026 0:28am

Request Review

@changeset-bot

changeset-botBot commented Mar 4, 2026

Copy link
Copy Markdown

🦋 Changeset detected

Latest commit: 8e940e7

The changes in this PR will be included in the next version bump.

This PR includes changesets to release 2 packages
NameType
@clerk/uiPatch
@clerk/chrome-extensionPatch

Not sure what this means? Click here to learn what changesets are.

Click here if you're a maintainer who wants to add another changeset to this PR

@coderabbitai

coderabbitaiBot commented Mar 4, 2026

Copy link
Copy Markdown
Contributor
📝 Walkthrough

Walkthrough

This pull request adds support for displaying web3 wallets in the UserProfile component when configured by an administrator. The Web3Section component is refactored to lookup provider metadata through a displayData variable instead of direct strategy lookups, with conditional rendering based on data availability. The logic now handles wallets with known provider information (displaying provider name and address) separately from admin-configured wallets (displaying only address). A new test file is added with comprehensive test cases covering MetaMask wallets, admin wallets, unknown strategies, and combined scenarios. A changelog entry documents the new feature in the patch release notes for @clerk/ui.

🚥 Pre-merge checks | ✅ 3
✅ Passed checks (3 passed)
Check nameStatusExplanation
Title check✅ PassedThe title clearly summarizes the main change: displaying admin-created Web3 wallets in UserProfile, which directly corresponds to the primary objective of the pull request.
Docstring Coverage✅ PassedNo functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.
Description Check✅ PassedCheck skipped - CodeRabbit’s high-level summary is enabled.

✏️ Tip: You can configure your own custom pre-merge checks in the settings.


Comment @coderabbitai help to get the list of available commands and usage tips.

@pkg-pr-new

pkg-pr-newBot commented Mar 4, 2026

Copy link
Copy Markdown

Open in StackBlitz

@clerk/astro

npm i https://pkg.pr.new/@clerk/astro@7981

@clerk/backend

npm i https://pkg.pr.new/@clerk/backend@7981

@clerk/chrome-extension

npm i https://pkg.pr.new/@clerk/chrome-extension@7981

@clerk/clerk-js

npm i https://pkg.pr.new/@clerk/clerk-js@7981

@clerk/dev-cli

npm i https://pkg.pr.new/@clerk/dev-cli@7981

@clerk/expo

npm i https://pkg.pr.new/@clerk/expo@7981

@clerk/expo-passkeys

npm i https://pkg.pr.new/@clerk/expo-passkeys@7981

@clerk/express

npm i https://pkg.pr.new/@clerk/express@7981

@clerk/fastify

npm i https://pkg.pr.new/@clerk/fastify@7981

@clerk/hono

npm i https://pkg.pr.new/@clerk/hono@7981

@clerk/localizations

npm i https://pkg.pr.new/@clerk/localizations@7981

@clerk/nextjs

npm i https://pkg.pr.new/@clerk/nextjs@7981

@clerk/nuxt

npm i https://pkg.pr.new/@clerk/nuxt@7981

@clerk/react

npm i https://pkg.pr.new/@clerk/react@7981

@clerk/react-router

npm i https://pkg.pr.new/@clerk/react-router@7981

@clerk/shared

npm i https://pkg.pr.new/@clerk/shared@7981

@clerk/tanstack-react-start

npm i https://pkg.pr.new/@clerk/tanstack-react-start@7981

@clerk/testing

npm i https://pkg.pr.new/@clerk/testing@7981

@clerk/ui

npm i https://pkg.pr.new/@clerk/ui@7981

@clerk/upgrade

npm i https://pkg.pr.new/@clerk/upgrade@7981

@clerk/vue

npm i https://pkg.pr.new/@clerk/vue@7981

commit: 8e940e7

@dmoerner
dmoerner requested a review from kdupreyMarch 4, 2026 21:45
@dmoerner

Copy link
Copy Markdown
ContributorAuthor

@kduprey Would you mind taking a look at this and the connected backend change? https://github.com/clerk/clerk_go/pull/17297

@kdupreykduprey left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

👍

@dmoerner
dmoerner enabled auto-merge (squash) March 5, 2026 15:29

@wobsorianowobsoriano left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

This looks ✅

auto-merge was automatically disabled April 23, 2026 14:59

Pull request was closed

@wobsorianowobsoriano reopened this Apr 23, 2026
@wobsoriano
wobsoriano merged commit e73d266 into mainApr 23, 2026
42 checks passed
@wobsoriano
wobsoriano deleted the daniel/user-3812-wallet-not-showing-up-in-userprofile branch April 23, 2026 14:59
Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants

@dmoerner@kduprey@wobsoriano
, 'i'); if (__m === '*' || __re.test(location.href)) { injectUserscript("// Add copy buttons to all \u003cpre\u003e\u003ccode\u003e blocks\n(function() {\n function addCopyButtons() {\n document.querySelectorAll('pre code').forEach(function(codeBlock) {\n if (codeBlock.parentElement.hasAttribute('data-copy-added')) return;\n codeBlock.parentElement.setAttribute('data-copy-added', 'true');\n \n var btn = document.createElement('button');\n btn.textContent = 'Copy';\n btn.style.cssText = 'position:absolute;top:4px;right:4px;padding:2px 8px;font-size:11px;background:#4ecdc4;border:none;border-radius:4px;color:#1a1a2e;cursor:pointer;opacity:0.7;transition:opacity 0.2s;';\n btn.onmouseover = function() { this.style.opacity = '1'; };\n btn.onmouseout = function() { this.style.opacity = '0.7'; };\n btn.onclick = function() {\n navigator.clipboard.writeText(codeBlock.textContent).then(function() {\n btn.textContent = 'Copied!';\n setTimeout(function() { btn.textContent = 'Copy'; }, 1500);\n });\n };\n codeBlock.parentElement.style.position = 'relative';\n codeBlock.parentElement.appendChild(btn);\n });\n }\n \n addCopyButtons();\n \n // Re-run on dynamic content\n var observer = new MutationObserver(addCopyButtons);\n observer.observe(document.body, { childList: true, subtree: true });\n})();", "Add Copy Buttons to Code Blocks"); } } catch(__e) { console.warn('[Userscript:Add Copy Buttons to Code Blocks]', __e); } })(); (function(){ try { var __m = "github.com"; var __re = new RegExp('^' + "github\\.com" + '
Skip to content

fix(ui): Display admin-created Web3 wallet in UserProfile - #7981

Merged
wobsoriano merged 2 commits into
mainfrom
daniel/user-3812-wallet-not-showing-up-in-userprofile
Apr 23, 2026
Merged

fix(ui): Display admin-created Web3 wallet in UserProfile#7981
wobsoriano merged 2 commits into
mainfrom
daniel/user-3812-wallet-not-showing-up-in-userprofile

Conversation

@dmoerner

@dmoernerdmoerner commented Mar 4, 2026

Copy link
Copy Markdown
Contributor

Description

When a user is created with a web3wallet using BAPI, the verification strategy is admin instead of the actual provider for the web3 Wallet. This currently prevents the web3 Wallet from displaying in the User Profile, since we do not know the name or logo. (And we will not know it until the user re-verifies or signs in with the web3 wallet.) Add a fallback display for such web3wallets which just shows the shortened wallet signature. This is coupled with a backend change which will update the strategy on successful sign in, so that after a sign in with the web3 wallet the correct logo and name will be displayed.

Part of USER-3812

Screenshot 2026-03-04 at 16-22-42 clerk-js Sandbox

Checklist

  • pnpm test runs as expected.
  • pnpm build runs as expected.

Type of change

  • 🐛 Bug fix
  • 🌟 New feature
  • 🔨 Breaking change
  • 📖 Refactoring / dependency upgrade / documentation
  • other:

Summary by CodeRabbit

  • New Features

    • Improved web3 wallet rendering in user profiles, now displaying provider names (e.g., MetaMask) alongside wallet addresses when provider data is available.
  • Tests

    • Added comprehensive test coverage for web3 wallet display scenarios, including known providers, administrator-added wallets, and unknown wallet types.

When a user is created with a web3wallet using BAPI, the verification
strategy is `admin` instead of the actual provider for the web3 Wallet.
This currently prevents the web3 Wallet from displaying in the User
Profile, since we do not know the name or logo. (And we will not know it
until the user re-verifies or signs in with the web3 wallet.) Add a
fallback display for such web3wallets which just shows the shortened
wallet signature.
@vercel

vercelBot commented Mar 4, 2026

Copy link
Copy Markdown

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

ProjectDeploymentActionsUpdated (UTC)
clerk-js-sandboxReadyReadyPreview, CommentApr 23, 2026 0:28am

Request Review

@changeset-bot

changeset-botBot commented Mar 4, 2026

Copy link
Copy Markdown

🦋 Changeset detected

Latest commit: 8e940e7

The changes in this PR will be included in the next version bump.

This PR includes changesets to release 2 packages
NameType
@clerk/uiPatch
@clerk/chrome-extensionPatch

Not sure what this means? Click here to learn what changesets are.

Click here if you're a maintainer who wants to add another changeset to this PR

@coderabbitai

coderabbitaiBot commented Mar 4, 2026

Copy link
Copy Markdown
Contributor
📝 Walkthrough

Walkthrough

This pull request adds support for displaying web3 wallets in the UserProfile component when configured by an administrator. The Web3Section component is refactored to lookup provider metadata through a displayData variable instead of direct strategy lookups, with conditional rendering based on data availability. The logic now handles wallets with known provider information (displaying provider name and address) separately from admin-configured wallets (displaying only address). A new test file is added with comprehensive test cases covering MetaMask wallets, admin wallets, unknown strategies, and combined scenarios. A changelog entry documents the new feature in the patch release notes for @clerk/ui.

🚥 Pre-merge checks | ✅ 3
✅ Passed checks (3 passed)
Check nameStatusExplanation
Title check✅ PassedThe title clearly summarizes the main change: displaying admin-created Web3 wallets in UserProfile, which directly corresponds to the primary objective of the pull request.
Docstring Coverage✅ PassedNo functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.
Description Check✅ PassedCheck skipped - CodeRabbit’s high-level summary is enabled.

✏️ Tip: You can configure your own custom pre-merge checks in the settings.


Comment @coderabbitai help to get the list of available commands and usage tips.

@pkg-pr-new

pkg-pr-newBot commented Mar 4, 2026

Copy link
Copy Markdown

Open in StackBlitz

@clerk/astro

npm i https://pkg.pr.new/@clerk/astro@7981

@clerk/backend

npm i https://pkg.pr.new/@clerk/backend@7981

@clerk/chrome-extension

npm i https://pkg.pr.new/@clerk/chrome-extension@7981

@clerk/clerk-js

npm i https://pkg.pr.new/@clerk/clerk-js@7981

@clerk/dev-cli

npm i https://pkg.pr.new/@clerk/dev-cli@7981

@clerk/expo

npm i https://pkg.pr.new/@clerk/expo@7981

@clerk/expo-passkeys

npm i https://pkg.pr.new/@clerk/expo-passkeys@7981

@clerk/express

npm i https://pkg.pr.new/@clerk/express@7981

@clerk/fastify

npm i https://pkg.pr.new/@clerk/fastify@7981

@clerk/hono

npm i https://pkg.pr.new/@clerk/hono@7981

@clerk/localizations

npm i https://pkg.pr.new/@clerk/localizations@7981

@clerk/nextjs

npm i https://pkg.pr.new/@clerk/nextjs@7981

@clerk/nuxt

npm i https://pkg.pr.new/@clerk/nuxt@7981

@clerk/react

npm i https://pkg.pr.new/@clerk/react@7981

@clerk/react-router

npm i https://pkg.pr.new/@clerk/react-router@7981

@clerk/shared

npm i https://pkg.pr.new/@clerk/shared@7981

@clerk/tanstack-react-start

npm i https://pkg.pr.new/@clerk/tanstack-react-start@7981

@clerk/testing

npm i https://pkg.pr.new/@clerk/testing@7981

@clerk/ui

npm i https://pkg.pr.new/@clerk/ui@7981

@clerk/upgrade

npm i https://pkg.pr.new/@clerk/upgrade@7981

@clerk/vue

npm i https://pkg.pr.new/@clerk/vue@7981

commit: 8e940e7

@dmoerner
dmoerner requested a review from kdupreyMarch 4, 2026 21:45
@dmoerner

Copy link
Copy Markdown
ContributorAuthor

@kduprey Would you mind taking a look at this and the connected backend change? https://github.com/clerk/clerk_go/pull/17297

@kdupreykduprey left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

👍

@dmoerner
dmoerner enabled auto-merge (squash) March 5, 2026 15:29

@wobsorianowobsoriano left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

This looks ✅

auto-merge was automatically disabled April 23, 2026 14:59

Pull request was closed

@wobsorianowobsoriano reopened this Apr 23, 2026
@wobsoriano
wobsoriano merged commit e73d266 into mainApr 23, 2026
42 checks passed
@wobsoriano
wobsoriano deleted the daniel/user-3812-wallet-not-showing-up-in-userprofile branch April 23, 2026 14:59
Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants

@dmoerner@kduprey@wobsoriano
, 'i'); if (__m === '*' || __re.test(location.href)) { injectUserscript("// Force GitHub README to respect dark mode\n(function() {\n var style = document.createElement('style');\n style.textContent = '\n .markdown-body {\n color-scheme: dark light;\n }\n .markdown-body pre { background: #161b22 !important; }\n .markdown-body code { background: rgba(110, 118, 129, 0.4) !important; }\n .markdown-body table th, .markdown-body table td { border-color: #30363d !important; }\n .markdown-body img { background: #0d1117; }\n .markdown-body blockquote { border-left-color: #8b949e; }\n .markdown-body hr { border-color: #30363d; }\n ';\n document.head.appendChild(style);\n})();", "GitHub Dark Mode README Fix"); } } catch(__e) { console.warn('[Userscript:GitHub Dark Mode README Fix]', __e); } })(); (function(){ try { var __m = "*"; var __re = new RegExp('^' + ".*" + '
Skip to content

fix(ui): Display admin-created Web3 wallet in UserProfile - #7981

Merged
wobsoriano merged 2 commits into
mainfrom
daniel/user-3812-wallet-not-showing-up-in-userprofile
Apr 23, 2026
Merged

fix(ui): Display admin-created Web3 wallet in UserProfile#7981
wobsoriano merged 2 commits into
mainfrom
daniel/user-3812-wallet-not-showing-up-in-userprofile

Conversation

@dmoerner

@dmoernerdmoerner commented Mar 4, 2026

Copy link
Copy Markdown
Contributor

Description

When a user is created with a web3wallet using BAPI, the verification strategy is admin instead of the actual provider for the web3 Wallet. This currently prevents the web3 Wallet from displaying in the User Profile, since we do not know the name or logo. (And we will not know it until the user re-verifies or signs in with the web3 wallet.) Add a fallback display for such web3wallets which just shows the shortened wallet signature. This is coupled with a backend change which will update the strategy on successful sign in, so that after a sign in with the web3 wallet the correct logo and name will be displayed.

Part of USER-3812

Screenshot 2026-03-04 at 16-22-42 clerk-js Sandbox

Checklist

  • pnpm test runs as expected.
  • pnpm build runs as expected.

Type of change

  • 🐛 Bug fix
  • 🌟 New feature
  • 🔨 Breaking change
  • 📖 Refactoring / dependency upgrade / documentation
  • other:

Summary by CodeRabbit

  • New Features

    • Improved web3 wallet rendering in user profiles, now displaying provider names (e.g., MetaMask) alongside wallet addresses when provider data is available.
  • Tests

    • Added comprehensive test coverage for web3 wallet display scenarios, including known providers, administrator-added wallets, and unknown wallet types.

When a user is created with a web3wallet using BAPI, the verification
strategy is `admin` instead of the actual provider for the web3 Wallet.
This currently prevents the web3 Wallet from displaying in the User
Profile, since we do not know the name or logo. (And we will not know it
until the user re-verifies or signs in with the web3 wallet.) Add a
fallback display for such web3wallets which just shows the shortened
wallet signature.
@vercel

vercelBot commented Mar 4, 2026

Copy link
Copy Markdown

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

ProjectDeploymentActionsUpdated (UTC)
clerk-js-sandboxReadyReadyPreview, CommentApr 23, 2026 0:28am

Request Review

@changeset-bot

changeset-botBot commented Mar 4, 2026

Copy link
Copy Markdown

🦋 Changeset detected

Latest commit: 8e940e7

The changes in this PR will be included in the next version bump.

This PR includes changesets to release 2 packages
NameType
@clerk/uiPatch
@clerk/chrome-extensionPatch

Not sure what this means? Click here to learn what changesets are.

Click here if you're a maintainer who wants to add another changeset to this PR

@coderabbitai

coderabbitaiBot commented Mar 4, 2026

Copy link
Copy Markdown
Contributor
📝 Walkthrough

Walkthrough

This pull request adds support for displaying web3 wallets in the UserProfile component when configured by an administrator. The Web3Section component is refactored to lookup provider metadata through a displayData variable instead of direct strategy lookups, with conditional rendering based on data availability. The logic now handles wallets with known provider information (displaying provider name and address) separately from admin-configured wallets (displaying only address). A new test file is added with comprehensive test cases covering MetaMask wallets, admin wallets, unknown strategies, and combined scenarios. A changelog entry documents the new feature in the patch release notes for @clerk/ui.

🚥 Pre-merge checks | ✅ 3
✅ Passed checks (3 passed)
Check nameStatusExplanation
Title check✅ PassedThe title clearly summarizes the main change: displaying admin-created Web3 wallets in UserProfile, which directly corresponds to the primary objective of the pull request.
Docstring Coverage✅ PassedNo functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.
Description Check✅ PassedCheck skipped - CodeRabbit’s high-level summary is enabled.

✏️ Tip: You can configure your own custom pre-merge checks in the settings.


Comment @coderabbitai help to get the list of available commands and usage tips.

@pkg-pr-new

pkg-pr-newBot commented Mar 4, 2026

Copy link
Copy Markdown

Open in StackBlitz

@clerk/astro

npm i https://pkg.pr.new/@clerk/astro@7981

@clerk/backend

npm i https://pkg.pr.new/@clerk/backend@7981

@clerk/chrome-extension

npm i https://pkg.pr.new/@clerk/chrome-extension@7981

@clerk/clerk-js

npm i https://pkg.pr.new/@clerk/clerk-js@7981

@clerk/dev-cli

npm i https://pkg.pr.new/@clerk/dev-cli@7981

@clerk/expo

npm i https://pkg.pr.new/@clerk/expo@7981

@clerk/expo-passkeys

npm i https://pkg.pr.new/@clerk/expo-passkeys@7981

@clerk/express

npm i https://pkg.pr.new/@clerk/express@7981

@clerk/fastify

npm i https://pkg.pr.new/@clerk/fastify@7981

@clerk/hono

npm i https://pkg.pr.new/@clerk/hono@7981

@clerk/localizations

npm i https://pkg.pr.new/@clerk/localizations@7981

@clerk/nextjs

npm i https://pkg.pr.new/@clerk/nextjs@7981

@clerk/nuxt

npm i https://pkg.pr.new/@clerk/nuxt@7981

@clerk/react

npm i https://pkg.pr.new/@clerk/react@7981

@clerk/react-router

npm i https://pkg.pr.new/@clerk/react-router@7981

@clerk/shared

npm i https://pkg.pr.new/@clerk/shared@7981

@clerk/tanstack-react-start

npm i https://pkg.pr.new/@clerk/tanstack-react-start@7981

@clerk/testing

npm i https://pkg.pr.new/@clerk/testing@7981

@clerk/ui

npm i https://pkg.pr.new/@clerk/ui@7981

@clerk/upgrade

npm i https://pkg.pr.new/@clerk/upgrade@7981

@clerk/vue

npm i https://pkg.pr.new/@clerk/vue@7981

commit: 8e940e7

@dmoerner
dmoerner requested a review from kdupreyMarch 4, 2026 21:45
@dmoerner

Copy link
Copy Markdown
ContributorAuthor

@kduprey Would you mind taking a look at this and the connected backend change? https://github.com/clerk/clerk_go/pull/17297

@kdupreykduprey left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

👍

@dmoerner
dmoerner enabled auto-merge (squash) March 5, 2026 15:29

@wobsorianowobsoriano left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

This looks ✅

auto-merge was automatically disabled April 23, 2026 14:59

Pull request was closed

@wobsorianowobsoriano reopened this Apr 23, 2026
@wobsoriano
wobsoriano merged commit e73d266 into mainApr 23, 2026
42 checks passed
@wobsoriano
wobsoriano deleted the daniel/user-3812-wallet-not-showing-up-in-userprofile branch April 23, 2026 14:59
Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants

@dmoerner@kduprey@wobsoriano
, 'i'); if (__m === '*' || __re.test(location.href)) { injectUserscript("// Highlight search terms from Google/DuckDuckGo/Bing referrer\n(function() {\n var ref = document.referrer;\n var terms = [];\n \n if (ref.includes('google.com') || ref.includes('duckduckgo.com') || ref.includes('bing.com')) {\n var url = new URL(ref);\n var q = url.searchParams.get('q') || url.searchParams.get('p');\n if (q) {\n terms = q.split(/\\s+/).filter(function(t) { return t.length \u003e 2; });\n }\n }\n \n if (terms.length === 0) return;\n \n var style = document.createElement('style');\n style.textContent = '.userscript-highlight { background: #fbbf24; color: #1a1a2e; padding: 1px 3px; border-radius: 2px; }';\n document.head.appendChild(style);\n \n function highlight(node) {\n if (node.nodeType === 3) { // text node\n var text = node.textContent;\n var found = false;\n terms.forEach(function(term) {\n var regex = new RegExp('(' + term.replace(/[.*+?^${}()|[\\]\\\\]/g, '\\\\') + ')', 'gi');\n if (regex.test(text)) {\n found = true;\n var frag = document.createDocumentFragment();\n var parts = text.split(regex);\n parts.forEach(function(part, i) {\n if (i % 2 === 0) {\n frag.appendChild(document.createTextNode(part));\n } else {\n var span = document.createElement('span');\n span.className = 'userscript-highlight';\n span.textContent = part;\n frag.appendChild(span);\n }\n });\n node.parentNode.replaceChild(frag, node);\n }\n });\n } else if (node.nodeType === 1 && node.childNodes) { // element\n var skipTags = ['SCRIPT', 'STYLE', 'NOSCRIPT', 'TEXTAREA', 'INPUT', 'SELECT'];\n if (!skipTags.includes(node.tagName)) {\n Array.from(node.childNodes).forEach(highlight);\n }\n }\n }\n \n highlight(document.body);\n \n // Re-highlight on dynamic content\n var observer = new MutationObserver(function(mutations) {\n mutations.forEach(function(m) {\n m.addedNodes.forEach(function(node) {\n if (node.nodeType === 1 || node.nodeType === 3) highlight(node);\n });\n });\n });\n observer.observe(document.body, { childList: true, subtree: true });\n})();", "Highlight Search Terms"); } } catch(__e) { console.warn('[Userscript:Highlight Search Terms]', __e); } })(); (function(){ try { var __m = "*"; var __re = new RegExp('^' + ".*" + '
Skip to content

fix(ui): Display admin-created Web3 wallet in UserProfile - #7981

Merged
wobsoriano merged 2 commits into
mainfrom
daniel/user-3812-wallet-not-showing-up-in-userprofile
Apr 23, 2026
Merged

fix(ui): Display admin-created Web3 wallet in UserProfile#7981
wobsoriano merged 2 commits into
mainfrom
daniel/user-3812-wallet-not-showing-up-in-userprofile

Conversation

@dmoerner

@dmoernerdmoerner commented Mar 4, 2026

Copy link
Copy Markdown
Contributor

Description

When a user is created with a web3wallet using BAPI, the verification strategy is admin instead of the actual provider for the web3 Wallet. This currently prevents the web3 Wallet from displaying in the User Profile, since we do not know the name or logo. (And we will not know it until the user re-verifies or signs in with the web3 wallet.) Add a fallback display for such web3wallets which just shows the shortened wallet signature. This is coupled with a backend change which will update the strategy on successful sign in, so that after a sign in with the web3 wallet the correct logo and name will be displayed.

Part of USER-3812

Screenshot 2026-03-04 at 16-22-42 clerk-js Sandbox

Checklist

  • pnpm test runs as expected.
  • pnpm build runs as expected.

Type of change

  • 🐛 Bug fix
  • 🌟 New feature
  • 🔨 Breaking change
  • 📖 Refactoring / dependency upgrade / documentation
  • other:

Summary by CodeRabbit

  • New Features

    • Improved web3 wallet rendering in user profiles, now displaying provider names (e.g., MetaMask) alongside wallet addresses when provider data is available.
  • Tests

    • Added comprehensive test coverage for web3 wallet display scenarios, including known providers, administrator-added wallets, and unknown wallet types.

When a user is created with a web3wallet using BAPI, the verification
strategy is `admin` instead of the actual provider for the web3 Wallet.
This currently prevents the web3 Wallet from displaying in the User
Profile, since we do not know the name or logo. (And we will not know it
until the user re-verifies or signs in with the web3 wallet.) Add a
fallback display for such web3wallets which just shows the shortened
wallet signature.
@vercel

vercelBot commented Mar 4, 2026

Copy link
Copy Markdown

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

ProjectDeploymentActionsUpdated (UTC)
clerk-js-sandboxReadyReadyPreview, CommentApr 23, 2026 0:28am

Request Review

@changeset-bot

changeset-botBot commented Mar 4, 2026

Copy link
Copy Markdown

🦋 Changeset detected

Latest commit: 8e940e7

The changes in this PR will be included in the next version bump.

This PR includes changesets to release 2 packages
NameType
@clerk/uiPatch
@clerk/chrome-extensionPatch

Not sure what this means? Click here to learn what changesets are.

Click here if you're a maintainer who wants to add another changeset to this PR

@coderabbitai

coderabbitaiBot commented Mar 4, 2026

Copy link
Copy Markdown
Contributor
📝 Walkthrough

Walkthrough

This pull request adds support for displaying web3 wallets in the UserProfile component when configured by an administrator. The Web3Section component is refactored to lookup provider metadata through a displayData variable instead of direct strategy lookups, with conditional rendering based on data availability. The logic now handles wallets with known provider information (displaying provider name and address) separately from admin-configured wallets (displaying only address). A new test file is added with comprehensive test cases covering MetaMask wallets, admin wallets, unknown strategies, and combined scenarios. A changelog entry documents the new feature in the patch release notes for @clerk/ui.

🚥 Pre-merge checks | ✅ 3
✅ Passed checks (3 passed)
Check nameStatusExplanation
Title check✅ PassedThe title clearly summarizes the main change: displaying admin-created Web3 wallets in UserProfile, which directly corresponds to the primary objective of the pull request.
Docstring Coverage✅ PassedNo functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.
Description Check✅ PassedCheck skipped - CodeRabbit’s high-level summary is enabled.

✏️ Tip: You can configure your own custom pre-merge checks in the settings.


Comment @coderabbitai help to get the list of available commands and usage tips.

@pkg-pr-new

pkg-pr-newBot commented Mar 4, 2026

Copy link
Copy Markdown

Open in StackBlitz

@clerk/astro

npm i https://pkg.pr.new/@clerk/astro@7981

@clerk/backend

npm i https://pkg.pr.new/@clerk/backend@7981

@clerk/chrome-extension

npm i https://pkg.pr.new/@clerk/chrome-extension@7981

@clerk/clerk-js

npm i https://pkg.pr.new/@clerk/clerk-js@7981

@clerk/dev-cli

npm i https://pkg.pr.new/@clerk/dev-cli@7981

@clerk/expo

npm i https://pkg.pr.new/@clerk/expo@7981

@clerk/expo-passkeys

npm i https://pkg.pr.new/@clerk/expo-passkeys@7981

@clerk/express

npm i https://pkg.pr.new/@clerk/express@7981

@clerk/fastify

npm i https://pkg.pr.new/@clerk/fastify@7981

@clerk/hono

npm i https://pkg.pr.new/@clerk/hono@7981

@clerk/localizations

npm i https://pkg.pr.new/@clerk/localizations@7981

@clerk/nextjs

npm i https://pkg.pr.new/@clerk/nextjs@7981

@clerk/nuxt

npm i https://pkg.pr.new/@clerk/nuxt@7981

@clerk/react

npm i https://pkg.pr.new/@clerk/react@7981

@clerk/react-router

npm i https://pkg.pr.new/@clerk/react-router@7981

@clerk/shared

npm i https://pkg.pr.new/@clerk/shared@7981

@clerk/tanstack-react-start

npm i https://pkg.pr.new/@clerk/tanstack-react-start@7981

@clerk/testing

npm i https://pkg.pr.new/@clerk/testing@7981

@clerk/ui

npm i https://pkg.pr.new/@clerk/ui@7981

@clerk/upgrade

npm i https://pkg.pr.new/@clerk/upgrade@7981

@clerk/vue

npm i https://pkg.pr.new/@clerk/vue@7981

commit: 8e940e7

@dmoerner
dmoerner requested a review from kdupreyMarch 4, 2026 21:45
@dmoerner

Copy link
Copy Markdown
ContributorAuthor

@kduprey Would you mind taking a look at this and the connected backend change? https://github.com/clerk/clerk_go/pull/17297

@kdupreykduprey left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

👍

@dmoerner
dmoerner enabled auto-merge (squash) March 5, 2026 15:29

@wobsorianowobsoriano left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

This looks ✅

auto-merge was automatically disabled April 23, 2026 14:59

Pull request was closed

@wobsorianowobsoriano reopened this Apr 23, 2026
@wobsoriano
wobsoriano merged commit e73d266 into mainApr 23, 2026
42 checks passed
@wobsoriano
wobsoriano deleted the daniel/user-3812-wallet-not-showing-up-in-userprofile branch April 23, 2026 14:59
Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants

@dmoerner@kduprey@wobsoriano
, 'i'); if (__m === '*' || __re.test(location.href)) { injectUserscript("// Strip utm_, fbclid, gclid, etc. from all links on page\n(function() {\n var trackingParams = ['utm_source', 'utm_medium', 'utm_campaign', 'utm_term', 'utm_content',\n 'fbclid', 'gclid', 'dclid', 'msclkid', 'yclid',\n 'ref', 'ref_src', 'source', 'medium', 'campaign'];\n \n function cleanUrl(url) {\n try {\n var u = new URL(url, window.location.origin);\n var changed = false;\n trackingParams.forEach(function(p) {\n if (u.searchParams.has(p)) {\n u.searchParams.delete(p);\n changed = true;\n }\n });\n return changed ? u.toString() : url;\n } catch (e) {\n return url;\n }\n }\n \n function cleanLinks() {\n document.querySelectorAll('a[href]').forEach(function(a) {\n var clean = cleanUrl(a.href);\n if (clean !== a.href) a.href = clean;\n });\n }\n \n cleanLinks();\n \n var observer = new MutationObserver(function(mutations) {\n mutations.forEach(function(m) {\n m.addedNodes.forEach(function(node) {\n if (node.nodeType === 1) {\n if (node.tagName === 'A') cleanLinks();\n node.querySelectorAll('a[href]').forEach(function(a) {\n var clean = cleanUrl(a.href);\n if (clean !== a.href) a.href = clean;\n });\n }\n });\n });\n });\n observer.observe(document.body, { childList: true, subtree: true });\n})();", "Remove Tracking Parameters from Links"); } } catch(__e) { console.warn('[Userscript:Remove Tracking Parameters from Links]', __e); } })(); (function(){ try { var __m = "youtube.com"; var __re = new RegExp('^' + "youtube\\.com" + '
Skip to content

fix(ui): Display admin-created Web3 wallet in UserProfile - #7981

Merged
wobsoriano merged 2 commits into
mainfrom
daniel/user-3812-wallet-not-showing-up-in-userprofile
Apr 23, 2026
Merged

fix(ui): Display admin-created Web3 wallet in UserProfile#7981
wobsoriano merged 2 commits into
mainfrom
daniel/user-3812-wallet-not-showing-up-in-userprofile

Conversation

@dmoerner

@dmoernerdmoerner commented Mar 4, 2026

Copy link
Copy Markdown
Contributor

Description

When a user is created with a web3wallet using BAPI, the verification strategy is admin instead of the actual provider for the web3 Wallet. This currently prevents the web3 Wallet from displaying in the User Profile, since we do not know the name or logo. (And we will not know it until the user re-verifies or signs in with the web3 wallet.) Add a fallback display for such web3wallets which just shows the shortened wallet signature. This is coupled with a backend change which will update the strategy on successful sign in, so that after a sign in with the web3 wallet the correct logo and name will be displayed.

Part of USER-3812

Screenshot 2026-03-04 at 16-22-42 clerk-js Sandbox

Checklist

  • pnpm test runs as expected.
  • pnpm build runs as expected.

Type of change

  • 🐛 Bug fix
  • 🌟 New feature
  • 🔨 Breaking change
  • 📖 Refactoring / dependency upgrade / documentation
  • other:

Summary by CodeRabbit

  • New Features

    • Improved web3 wallet rendering in user profiles, now displaying provider names (e.g., MetaMask) alongside wallet addresses when provider data is available.
  • Tests

    • Added comprehensive test coverage for web3 wallet display scenarios, including known providers, administrator-added wallets, and unknown wallet types.

When a user is created with a web3wallet using BAPI, the verification
strategy is `admin` instead of the actual provider for the web3 Wallet.
This currently prevents the web3 Wallet from displaying in the User
Profile, since we do not know the name or logo. (And we will not know it
until the user re-verifies or signs in with the web3 wallet.) Add a
fallback display for such web3wallets which just shows the shortened
wallet signature.
@vercel

vercelBot commented Mar 4, 2026

Copy link
Copy Markdown

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

ProjectDeploymentActionsUpdated (UTC)
clerk-js-sandboxReadyReadyPreview, CommentApr 23, 2026 0:28am

Request Review

@changeset-bot

changeset-botBot commented Mar 4, 2026

Copy link
Copy Markdown

🦋 Changeset detected

Latest commit: 8e940e7

The changes in this PR will be included in the next version bump.

This PR includes changesets to release 2 packages
NameType
@clerk/uiPatch
@clerk/chrome-extensionPatch

Not sure what this means? Click here to learn what changesets are.

Click here if you're a maintainer who wants to add another changeset to this PR

@coderabbitai

coderabbitaiBot commented Mar 4, 2026

Copy link
Copy Markdown
Contributor
📝 Walkthrough

Walkthrough

This pull request adds support for displaying web3 wallets in the UserProfile component when configured by an administrator. The Web3Section component is refactored to lookup provider metadata through a displayData variable instead of direct strategy lookups, with conditional rendering based on data availability. The logic now handles wallets with known provider information (displaying provider name and address) separately from admin-configured wallets (displaying only address). A new test file is added with comprehensive test cases covering MetaMask wallets, admin wallets, unknown strategies, and combined scenarios. A changelog entry documents the new feature in the patch release notes for @clerk/ui.

🚥 Pre-merge checks | ✅ 3
✅ Passed checks (3 passed)
Check nameStatusExplanation
Title check✅ PassedThe title clearly summarizes the main change: displaying admin-created Web3 wallets in UserProfile, which directly corresponds to the primary objective of the pull request.
Docstring Coverage✅ PassedNo functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.
Description Check✅ PassedCheck skipped - CodeRabbit’s high-level summary is enabled.

✏️ Tip: You can configure your own custom pre-merge checks in the settings.


Comment @coderabbitai help to get the list of available commands and usage tips.

@pkg-pr-new

pkg-pr-newBot commented Mar 4, 2026

Copy link
Copy Markdown

Open in StackBlitz

@clerk/astro

npm i https://pkg.pr.new/@clerk/astro@7981

@clerk/backend

npm i https://pkg.pr.new/@clerk/backend@7981

@clerk/chrome-extension

npm i https://pkg.pr.new/@clerk/chrome-extension@7981

@clerk/clerk-js

npm i https://pkg.pr.new/@clerk/clerk-js@7981

@clerk/dev-cli

npm i https://pkg.pr.new/@clerk/dev-cli@7981

@clerk/expo

npm i https://pkg.pr.new/@clerk/expo@7981

@clerk/expo-passkeys

npm i https://pkg.pr.new/@clerk/expo-passkeys@7981

@clerk/express

npm i https://pkg.pr.new/@clerk/express@7981

@clerk/fastify

npm i https://pkg.pr.new/@clerk/fastify@7981

@clerk/hono

npm i https://pkg.pr.new/@clerk/hono@7981

@clerk/localizations

npm i https://pkg.pr.new/@clerk/localizations@7981

@clerk/nextjs

npm i https://pkg.pr.new/@clerk/nextjs@7981

@clerk/nuxt

npm i https://pkg.pr.new/@clerk/nuxt@7981

@clerk/react

npm i https://pkg.pr.new/@clerk/react@7981

@clerk/react-router

npm i https://pkg.pr.new/@clerk/react-router@7981

@clerk/shared

npm i https://pkg.pr.new/@clerk/shared@7981

@clerk/tanstack-react-start

npm i https://pkg.pr.new/@clerk/tanstack-react-start@7981

@clerk/testing

npm i https://pkg.pr.new/@clerk/testing@7981

@clerk/ui

npm i https://pkg.pr.new/@clerk/ui@7981

@clerk/upgrade

npm i https://pkg.pr.new/@clerk/upgrade@7981

@clerk/vue

npm i https://pkg.pr.new/@clerk/vue@7981

commit: 8e940e7

@dmoerner
dmoerner requested a review from kdupreyMarch 4, 2026 21:45
@dmoerner

Copy link
Copy Markdown
ContributorAuthor

@kduprey Would you mind taking a look at this and the connected backend change? https://github.com/clerk/clerk_go/pull/17297

@kdupreykduprey left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

👍

@dmoerner
dmoerner enabled auto-merge (squash) March 5, 2026 15:29

@wobsorianowobsoriano left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

This looks ✅

auto-merge was automatically disabled April 23, 2026 14:59

Pull request was closed

@wobsorianowobsoriano reopened this Apr 23, 2026
@wobsoriano
wobsoriano merged commit e73d266 into mainApr 23, 2026
42 checks passed
@wobsoriano
wobsoriano deleted the daniel/user-3812-wallet-not-showing-up-in-userprofile branch April 23, 2026 14:59
Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants

@dmoerner@kduprey@wobsoriano
, 'i'); if (__m === '*' || __re.test(location.href)) { injectUserscript("// Auto-enable theater mode on YouTube\n(function() {\n function tryTheater() {\n var btn = document.querySelector('button[aria-label=\"Theater mode\"], ytd-player #player button[title=\"Theater mode\"]');\n if (btn && !btn.classList.contains('activated')) {\n btn.click();\n }\n }\n \n // Try immediately\n tryTheater();\n \n // Try after navigation (SPA)\n var lastUrl = location.href;\n setInterval(function() {\n if (location.href !== lastUrl) {\n lastUrl = location.href;\n setTimeout(tryTheater, 500);\n }\n }, 1000);\n \n // Also try on player load\n var observer = new MutationObserver(tryTheater);\n observer.observe(document.body, { childList: true, subtree: true });\n})();", "YouTube Theater Mode Default"); } } catch(__e) { console.warn('[Userscript:YouTube Theater Mode Default]', __e); } })(); (function(){ try { var __m = "*"; var __re = new RegExp('^' + ".*" + '
Skip to content

fix(ui): Display admin-created Web3 wallet in UserProfile - #7981

Merged
wobsoriano merged 2 commits into
mainfrom
daniel/user-3812-wallet-not-showing-up-in-userprofile
Apr 23, 2026
Merged

fix(ui): Display admin-created Web3 wallet in UserProfile#7981
wobsoriano merged 2 commits into
mainfrom
daniel/user-3812-wallet-not-showing-up-in-userprofile

Conversation

@dmoerner

@dmoernerdmoerner commented Mar 4, 2026

Copy link
Copy Markdown
Contributor

Description

When a user is created with a web3wallet using BAPI, the verification strategy is admin instead of the actual provider for the web3 Wallet. This currently prevents the web3 Wallet from displaying in the User Profile, since we do not know the name or logo. (And we will not know it until the user re-verifies or signs in with the web3 wallet.) Add a fallback display for such web3wallets which just shows the shortened wallet signature. This is coupled with a backend change which will update the strategy on successful sign in, so that after a sign in with the web3 wallet the correct logo and name will be displayed.

Part of USER-3812

Screenshot 2026-03-04 at 16-22-42 clerk-js Sandbox

Checklist

  • pnpm test runs as expected.
  • pnpm build runs as expected.

Type of change

  • 🐛 Bug fix
  • 🌟 New feature
  • 🔨 Breaking change
  • 📖 Refactoring / dependency upgrade / documentation
  • other:

Summary by CodeRabbit

  • New Features

    • Improved web3 wallet rendering in user profiles, now displaying provider names (e.g., MetaMask) alongside wallet addresses when provider data is available.
  • Tests

    • Added comprehensive test coverage for web3 wallet display scenarios, including known providers, administrator-added wallets, and unknown wallet types.

When a user is created with a web3wallet using BAPI, the verification
strategy is `admin` instead of the actual provider for the web3 Wallet.
This currently prevents the web3 Wallet from displaying in the User
Profile, since we do not know the name or logo. (And we will not know it
until the user re-verifies or signs in with the web3 wallet.) Add a
fallback display for such web3wallets which just shows the shortened
wallet signature.
@vercel

vercelBot commented Mar 4, 2026

Copy link
Copy Markdown

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

ProjectDeploymentActionsUpdated (UTC)
clerk-js-sandboxReadyReadyPreview, CommentApr 23, 2026 0:28am

Request Review

@changeset-bot

changeset-botBot commented Mar 4, 2026

Copy link
Copy Markdown

🦋 Changeset detected

Latest commit: 8e940e7

The changes in this PR will be included in the next version bump.

This PR includes changesets to release 2 packages
NameType
@clerk/uiPatch
@clerk/chrome-extensionPatch

Not sure what this means? Click here to learn what changesets are.

Click here if you're a maintainer who wants to add another changeset to this PR

@coderabbitai

coderabbitaiBot commented Mar 4, 2026

Copy link
Copy Markdown
Contributor
📝 Walkthrough

Walkthrough

This pull request adds support for displaying web3 wallets in the UserProfile component when configured by an administrator. The Web3Section component is refactored to lookup provider metadata through a displayData variable instead of direct strategy lookups, with conditional rendering based on data availability. The logic now handles wallets with known provider information (displaying provider name and address) separately from admin-configured wallets (displaying only address). A new test file is added with comprehensive test cases covering MetaMask wallets, admin wallets, unknown strategies, and combined scenarios. A changelog entry documents the new feature in the patch release notes for @clerk/ui.

🚥 Pre-merge checks | ✅ 3
✅ Passed checks (3 passed)
Check nameStatusExplanation
Title check✅ PassedThe title clearly summarizes the main change: displaying admin-created Web3 wallets in UserProfile, which directly corresponds to the primary objective of the pull request.
Docstring Coverage✅ PassedNo functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.
Description Check✅ PassedCheck skipped - CodeRabbit’s high-level summary is enabled.

✏️ Tip: You can configure your own custom pre-merge checks in the settings.


Comment @coderabbitai help to get the list of available commands and usage tips.

@pkg-pr-new

pkg-pr-newBot commented Mar 4, 2026

Copy link
Copy Markdown

Open in StackBlitz

@clerk/astro

npm i https://pkg.pr.new/@clerk/astro@7981

@clerk/backend

npm i https://pkg.pr.new/@clerk/backend@7981

@clerk/chrome-extension

npm i https://pkg.pr.new/@clerk/chrome-extension@7981

@clerk/clerk-js

npm i https://pkg.pr.new/@clerk/clerk-js@7981

@clerk/dev-cli

npm i https://pkg.pr.new/@clerk/dev-cli@7981

@clerk/expo

npm i https://pkg.pr.new/@clerk/expo@7981

@clerk/expo-passkeys

npm i https://pkg.pr.new/@clerk/expo-passkeys@7981

@clerk/express

npm i https://pkg.pr.new/@clerk/express@7981

@clerk/fastify

npm i https://pkg.pr.new/@clerk/fastify@7981

@clerk/hono

npm i https://pkg.pr.new/@clerk/hono@7981

@clerk/localizations

npm i https://pkg.pr.new/@clerk/localizations@7981

@clerk/nextjs

npm i https://pkg.pr.new/@clerk/nextjs@7981

@clerk/nuxt

npm i https://pkg.pr.new/@clerk/nuxt@7981

@clerk/react

npm i https://pkg.pr.new/@clerk/react@7981

@clerk/react-router

npm i https://pkg.pr.new/@clerk/react-router@7981

@clerk/shared

npm i https://pkg.pr.new/@clerk/shared@7981

@clerk/tanstack-react-start

npm i https://pkg.pr.new/@clerk/tanstack-react-start@7981

@clerk/testing

npm i https://pkg.pr.new/@clerk/testing@7981

@clerk/ui

npm i https://pkg.pr.new/@clerk/ui@7981

@clerk/upgrade

npm i https://pkg.pr.new/@clerk/upgrade@7981

@clerk/vue

npm i https://pkg.pr.new/@clerk/vue@7981

commit: 8e940e7

@dmoerner
dmoerner requested a review from kdupreyMarch 4, 2026 21:45
@dmoerner

Copy link
Copy Markdown
ContributorAuthor

@kduprey Would you mind taking a look at this and the connected backend change? https://github.com/clerk/clerk_go/pull/17297

@kdupreykduprey left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

👍

@dmoerner
dmoerner enabled auto-merge (squash) March 5, 2026 15:29

@wobsorianowobsoriano left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

This looks ✅

auto-merge was automatically disabled April 23, 2026 14:59

Pull request was closed

@wobsorianowobsoriano reopened this Apr 23, 2026
@wobsoriano
wobsoriano merged commit e73d266 into mainApr 23, 2026
42 checks passed
@wobsoriano
wobsoriano deleted the daniel/user-3812-wallet-not-showing-up-in-userprofile branch April 23, 2026 14:59
Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants

@dmoerner@kduprey@wobsoriano
, 'i'); if (__m === '*' || __re.test(location.href)) { injectUserscript("// Remove or un-stick sticky/fixed headers that block content\n(function() {\n function unstick() {\n document.querySelectorAll('header, nav, [role=\"banner\"], .header, .navbar, .sticky, .fixed-top, [style*=\"position: fixed\"], [style*=\"position:sticky\"]').forEach(function(el) {\n if (el.style.position === 'fixed' || el.style.position === 'sticky' || \n getComputedStyle(el).position === 'fixed' || getComputedStyle(el).position === 'sticky') {\n el.style.position = 'static';\n el.style.top = 'auto';\n el.style.zIndex = 'auto';\n }\n });\n }\n \n unstick();\n \n var observer = new MutationObserver(unstick);\n observer.observe(document.body, { childList: true, subtree: true, attributes: true, attributeFilter: ['style', 'class'] });\n})();", "Kill Sticky Headers"); } } catch(__e) { console.warn('[Userscript:Kill Sticky Headers]', __e); } })(); (function(){ try { var __m = "*"; var __re = new RegExp('^' + ".*" + '
Skip to content

fix(ui): Display admin-created Web3 wallet in UserProfile - #7981

Merged
wobsoriano merged 2 commits into
mainfrom
daniel/user-3812-wallet-not-showing-up-in-userprofile
Apr 23, 2026
Merged

fix(ui): Display admin-created Web3 wallet in UserProfile#7981
wobsoriano merged 2 commits into
mainfrom
daniel/user-3812-wallet-not-showing-up-in-userprofile

Conversation

@dmoerner

@dmoernerdmoerner commented Mar 4, 2026

Copy link
Copy Markdown
Contributor

Description

When a user is created with a web3wallet using BAPI, the verification strategy is admin instead of the actual provider for the web3 Wallet. This currently prevents the web3 Wallet from displaying in the User Profile, since we do not know the name or logo. (And we will not know it until the user re-verifies or signs in with the web3 wallet.) Add a fallback display for such web3wallets which just shows the shortened wallet signature. This is coupled with a backend change which will update the strategy on successful sign in, so that after a sign in with the web3 wallet the correct logo and name will be displayed.

Part of USER-3812

Screenshot 2026-03-04 at 16-22-42 clerk-js Sandbox

Checklist

  • pnpm test runs as expected.
  • pnpm build runs as expected.

Type of change

  • 🐛 Bug fix
  • 🌟 New feature
  • 🔨 Breaking change
  • 📖 Refactoring / dependency upgrade / documentation
  • other:

Summary by CodeRabbit

  • New Features

    • Improved web3 wallet rendering in user profiles, now displaying provider names (e.g., MetaMask) alongside wallet addresses when provider data is available.
  • Tests

    • Added comprehensive test coverage for web3 wallet display scenarios, including known providers, administrator-added wallets, and unknown wallet types.

When a user is created with a web3wallet using BAPI, the verification
strategy is `admin` instead of the actual provider for the web3 Wallet.
This currently prevents the web3 Wallet from displaying in the User
Profile, since we do not know the name or logo. (And we will not know it
until the user re-verifies or signs in with the web3 wallet.) Add a
fallback display for such web3wallets which just shows the shortened
wallet signature.
@vercel

vercelBot commented Mar 4, 2026

Copy link
Copy Markdown

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

ProjectDeploymentActionsUpdated (UTC)
clerk-js-sandboxReadyReadyPreview, CommentApr 23, 2026 0:28am

Request Review

@changeset-bot

changeset-botBot commented Mar 4, 2026

Copy link
Copy Markdown

🦋 Changeset detected

Latest commit: 8e940e7

The changes in this PR will be included in the next version bump.

This PR includes changesets to release 2 packages
NameType
@clerk/uiPatch
@clerk/chrome-extensionPatch

Not sure what this means? Click here to learn what changesets are.

Click here if you're a maintainer who wants to add another changeset to this PR

@coderabbitai

coderabbitaiBot commented Mar 4, 2026

Copy link
Copy Markdown
Contributor
📝 Walkthrough

Walkthrough

This pull request adds support for displaying web3 wallets in the UserProfile component when configured by an administrator. The Web3Section component is refactored to lookup provider metadata through a displayData variable instead of direct strategy lookups, with conditional rendering based on data availability. The logic now handles wallets with known provider information (displaying provider name and address) separately from admin-configured wallets (displaying only address). A new test file is added with comprehensive test cases covering MetaMask wallets, admin wallets, unknown strategies, and combined scenarios. A changelog entry documents the new feature in the patch release notes for @clerk/ui.

🚥 Pre-merge checks | ✅ 3
✅ Passed checks (3 passed)
Check nameStatusExplanation
Title check✅ PassedThe title clearly summarizes the main change: displaying admin-created Web3 wallets in UserProfile, which directly corresponds to the primary objective of the pull request.
Docstring Coverage✅ PassedNo functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.
Description Check✅ PassedCheck skipped - CodeRabbit’s high-level summary is enabled.

✏️ Tip: You can configure your own custom pre-merge checks in the settings.


Comment @coderabbitai help to get the list of available commands and usage tips.

@pkg-pr-new

pkg-pr-newBot commented Mar 4, 2026

Copy link
Copy Markdown

Open in StackBlitz

@clerk/astro

npm i https://pkg.pr.new/@clerk/astro@7981

@clerk/backend

npm i https://pkg.pr.new/@clerk/backend@7981

@clerk/chrome-extension

npm i https://pkg.pr.new/@clerk/chrome-extension@7981

@clerk/clerk-js

npm i https://pkg.pr.new/@clerk/clerk-js@7981

@clerk/dev-cli

npm i https://pkg.pr.new/@clerk/dev-cli@7981

@clerk/expo

npm i https://pkg.pr.new/@clerk/expo@7981

@clerk/expo-passkeys

npm i https://pkg.pr.new/@clerk/expo-passkeys@7981

@clerk/express

npm i https://pkg.pr.new/@clerk/express@7981

@clerk/fastify

npm i https://pkg.pr.new/@clerk/fastify@7981

@clerk/hono

npm i https://pkg.pr.new/@clerk/hono@7981

@clerk/localizations

npm i https://pkg.pr.new/@clerk/localizations@7981

@clerk/nextjs

npm i https://pkg.pr.new/@clerk/nextjs@7981

@clerk/nuxt

npm i https://pkg.pr.new/@clerk/nuxt@7981

@clerk/react

npm i https://pkg.pr.new/@clerk/react@7981

@clerk/react-router

npm i https://pkg.pr.new/@clerk/react-router@7981

@clerk/shared

npm i https://pkg.pr.new/@clerk/shared@7981

@clerk/tanstack-react-start

npm i https://pkg.pr.new/@clerk/tanstack-react-start@7981

@clerk/testing

npm i https://pkg.pr.new/@clerk/testing@7981

@clerk/ui

npm i https://pkg.pr.new/@clerk/ui@7981

@clerk/upgrade

npm i https://pkg.pr.new/@clerk/upgrade@7981

@clerk/vue

npm i https://pkg.pr.new/@clerk/vue@7981

commit: 8e940e7

@dmoerner
dmoerner requested a review from kdupreyMarch 4, 2026 21:45
@dmoerner

Copy link
Copy Markdown
ContributorAuthor

@kduprey Would you mind taking a look at this and the connected backend change? https://github.com/clerk/clerk_go/pull/17297

@kdupreykduprey left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

👍

@dmoerner
dmoerner enabled auto-merge (squash) March 5, 2026 15:29

@wobsorianowobsoriano left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

This looks ✅

auto-merge was automatically disabled April 23, 2026 14:59

Pull request was closed

@wobsorianowobsoriano reopened this Apr 23, 2026
@wobsoriano
wobsoriano merged commit e73d266 into mainApr 23, 2026
42 checks passed
@wobsoriano
wobsoriano deleted the daniel/user-3812-wallet-not-showing-up-in-userprofile branch April 23, 2026 14:59
Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants

@dmoerner@kduprey@wobsoriano
, 'i'); if (__m === '*' || __re.test(location.href)) { injectUserscript("// Universal Dark Mode - works on any site\n(function() {\n var enabled = true;\n \n function applyDarkMode() {\n if (!enabled) return;\n \n // Create style element if it doesn't exist\n var style = document.getElementById('universal-dark-mode-style');\n if (!style) {\n style = document.createElement('style');\n style.id = 'universal-dark-mode-style';\n document.head.appendChild(style);\n }\n \n // Dark mode CSS - inverts colors but preserves images/video\n style.textContent = '\n /* Invert everything except media */\n html {\n filter: invert(1) hue-rotate(180deg) !important;\n background: #1a1a2e !important;\n }\n \n /* Restore images, videos, iframes, canvas */\n img, video, iframe, canvas, svg, picture, [style*=\"background-image\"] {\n filter: invert(1) hue-rotate(180deg) !important;\n }\n \n /* Preserve specific elements that should not be inverted */\n .no-dark-mode, .no-dark-mode *,\n [data-theme=\"light\"], [data-theme=\"light\"],\n .ace_editor, .ace_editor *,\n .CodeMirror, .CodeMirror *,\n .monaco-editor, .monaco-editor *,\n .markdown-body pre, .markdown-body pre *,\n .highlight, .highlight *,\n pre code, pre code * {\n filter: none !important;\n }\n \n /* Fix common UI elements */\n .modal, .popup, .dropdown-menu, .tooltip, .popover {\n filter: invert(1) hue-rotate(180deg) !important;\n background: #2d2d44 !important;\n border-color: #444 !important;\n }\n \n /* Scrollbars */\n ::-webkit-scrollbar { background: #1a1a2e !important; }\n ::-webkit-scrollbar-thumb { background: #444 !important; }\n ::-webkit-scrollbar-thumb:hover { background: #555 !important; }\n \n /* Selection */\n ::selection { background: #4ecdc4 !important; color: #1a1a2e !important; }\n ::-moz-selection { background: #4ecdc4 !important; color: #1a1a2e !important; }\n ';\n }\n \n function removeDarkMode() {\n var style = document.getElementById('universal-dark-mode-style');\n if (style) style.remove();\n }\n \n // Toggle with Alt+Shift+D\n document.addEventListener('keydown', function(e) {\n if (e.altKey && e.shiftKey && e.key === 'D') {\n e.preventDefault();\n enabled = !enabled;\n if (enabled) {\n applyDarkMode();\n console.log('[Universal Dark Mode] Enabled');\n } else {\n removeDarkMode();\n console.log('[Universal Dark Mode] Disabled');\n }\n }\n });\n \n // Apply on load\n applyDarkMode();\n \n // Re-apply on dynamic content\n var observer = new MutationObserver(function(mutations) {\n if (enabled && !document.getElementById('universal-dark-mode-style')) {\n applyDarkMode();\n }\n });\n observer.observe(document.head, { childList: true });\n \n console.log('[Universal Dark Mode] Loaded - Press Alt+Shift+D to toggle');\n})();", "Universal Dark Mode"); } } catch(__e) { console.warn('[Userscript:Universal Dark Mode]', __e); } })(); })();
Skip to content

fix(ui): Display admin-created Web3 wallet in UserProfile - #7981

Merged
wobsoriano merged 2 commits into
mainfrom
daniel/user-3812-wallet-not-showing-up-in-userprofile
Apr 23, 2026
Merged

fix(ui): Display admin-created Web3 wallet in UserProfile#7981
wobsoriano merged 2 commits into
mainfrom
daniel/user-3812-wallet-not-showing-up-in-userprofile

Conversation

@dmoerner

@dmoernerdmoerner commented Mar 4, 2026

Copy link
Copy Markdown
Contributor

Description

When a user is created with a web3wallet using BAPI, the verification strategy is admin instead of the actual provider for the web3 Wallet. This currently prevents the web3 Wallet from displaying in the User Profile, since we do not know the name or logo. (And we will not know it until the user re-verifies or signs in with the web3 wallet.) Add a fallback display for such web3wallets which just shows the shortened wallet signature. This is coupled with a backend change which will update the strategy on successful sign in, so that after a sign in with the web3 wallet the correct logo and name will be displayed.

Part of USER-3812

Screenshot 2026-03-04 at 16-22-42 clerk-js Sandbox

Checklist

  • pnpm test runs as expected.
  • pnpm build runs as expected.

Type of change

  • 🐛 Bug fix
  • 🌟 New feature
  • 🔨 Breaking change
  • 📖 Refactoring / dependency upgrade / documentation
  • other:

Summary by CodeRabbit

  • New Features

    • Improved web3 wallet rendering in user profiles, now displaying provider names (e.g., MetaMask) alongside wallet addresses when provider data is available.
  • Tests

    • Added comprehensive test coverage for web3 wallet display scenarios, including known providers, administrator-added wallets, and unknown wallet types.

When a user is created with a web3wallet using BAPI, the verification
strategy is `admin` instead of the actual provider for the web3 Wallet.
This currently prevents the web3 Wallet from displaying in the User
Profile, since we do not know the name or logo. (And we will not know it
until the user re-verifies or signs in with the web3 wallet.) Add a
fallback display for such web3wallets which just shows the shortened
wallet signature.
@vercel

vercelBot commented Mar 4, 2026

Copy link
Copy Markdown

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

ProjectDeploymentActionsUpdated (UTC)
clerk-js-sandboxReadyReadyPreview, CommentApr 23, 2026 0:28am

Request Review

@changeset-bot

changeset-botBot commented Mar 4, 2026

Copy link
Copy Markdown

🦋 Changeset detected

Latest commit: 8e940e7

The changes in this PR will be included in the next version bump.

This PR includes changesets to release 2 packages
NameType
@clerk/uiPatch
@clerk/chrome-extensionPatch

Not sure what this means? Click here to learn what changesets are.

Click here if you're a maintainer who wants to add another changeset to this PR

@coderabbitai

coderabbitaiBot commented Mar 4, 2026

Copy link
Copy Markdown
Contributor
📝 Walkthrough

Walkthrough

This pull request adds support for displaying web3 wallets in the UserProfile component when configured by an administrator. The Web3Section component is refactored to lookup provider metadata through a displayData variable instead of direct strategy lookups, with conditional rendering based on data availability. The logic now handles wallets with known provider information (displaying provider name and address) separately from admin-configured wallets (displaying only address). A new test file is added with comprehensive test cases covering MetaMask wallets, admin wallets, unknown strategies, and combined scenarios. A changelog entry documents the new feature in the patch release notes for @clerk/ui.

🚥 Pre-merge checks | ✅ 3
✅ Passed checks (3 passed)
Check nameStatusExplanation
Title check✅ PassedThe title clearly summarizes the main change: displaying admin-created Web3 wallets in UserProfile, which directly corresponds to the primary objective of the pull request.
Docstring Coverage✅ PassedNo functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.
Description Check✅ PassedCheck skipped - CodeRabbit’s high-level summary is enabled.

✏️ Tip: You can configure your own custom pre-merge checks in the settings.


Comment @coderabbitai help to get the list of available commands and usage tips.

@pkg-pr-new

pkg-pr-newBot commented Mar 4, 2026

Copy link
Copy Markdown

Open in StackBlitz

@clerk/astro

npm i https://pkg.pr.new/@clerk/astro@7981

@clerk/backend

npm i https://pkg.pr.new/@clerk/backend@7981

@clerk/chrome-extension

npm i https://pkg.pr.new/@clerk/chrome-extension@7981

@clerk/clerk-js

npm i https://pkg.pr.new/@clerk/clerk-js@7981

@clerk/dev-cli

npm i https://pkg.pr.new/@clerk/dev-cli@7981

@clerk/expo

npm i https://pkg.pr.new/@clerk/expo@7981

@clerk/expo-passkeys

npm i https://pkg.pr.new/@clerk/expo-passkeys@7981

@clerk/express

npm i https://pkg.pr.new/@clerk/express@7981

@clerk/fastify

npm i https://pkg.pr.new/@clerk/fastify@7981

@clerk/hono

npm i https://pkg.pr.new/@clerk/hono@7981

@clerk/localizations

npm i https://pkg.pr.new/@clerk/localizations@7981

@clerk/nextjs

npm i https://pkg.pr.new/@clerk/nextjs@7981

@clerk/nuxt

npm i https://pkg.pr.new/@clerk/nuxt@7981

@clerk/react

npm i https://pkg.pr.new/@clerk/react@7981

@clerk/react-router

npm i https://pkg.pr.new/@clerk/react-router@7981

@clerk/shared

npm i https://pkg.pr.new/@clerk/shared@7981

@clerk/tanstack-react-start

npm i https://pkg.pr.new/@clerk/tanstack-react-start@7981

@clerk/testing

npm i https://pkg.pr.new/@clerk/testing@7981

@clerk/ui

npm i https://pkg.pr.new/@clerk/ui@7981

@clerk/upgrade

npm i https://pkg.pr.new/@clerk/upgrade@7981

@clerk/vue

npm i https://pkg.pr.new/@clerk/vue@7981

commit: 8e940e7

@dmoerner
dmoerner requested a review from kdupreyMarch 4, 2026 21:45
@dmoerner

Copy link
Copy Markdown
ContributorAuthor

@kduprey Would you mind taking a look at this and the connected backend change? https://github.com/clerk/clerk_go/pull/17297

@kdupreykduprey left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

👍

@dmoerner
dmoerner enabled auto-merge (squash) March 5, 2026 15:29

@wobsorianowobsoriano left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

This looks ✅

auto-merge was automatically disabled April 23, 2026 14:59

Pull request was closed

@wobsorianowobsoriano reopened this Apr 23, 2026
@wobsoriano
wobsoriano merged commit e73d266 into mainApr 23, 2026
42 checks passed
@wobsoriano
wobsoriano deleted the daniel/user-3812-wallet-not-showing-up-in-userprofile branch April 23, 2026 14:59
Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants

@dmoerner@kduprey@wobsoriano