Skip to content

Restrict Robinhood Wallet to supported chains (INT2-991) - #1089

Merged
pedromcunha merged 1 commit into
mainfrom
pedro/int2-991-set-wallet-restrictions-on-robinhood-wallet
Jul 24, 2026
Merged

Restrict Robinhood Wallet to supported chains (INT2-991)#1089
pedromcunha merged 1 commit into
mainfrom
pedro/int2-991-set-wallet-restrictions-on-robinhood-wallet

Conversation

@pedromcunha

Copy link
Copy Markdown
Member

No description provided.

@pedromcunha
pedromcunha requested a review from ted-palmer July 24, 2026 18:50
@vercel

vercel Bot commented Jul 24, 2026

Copy link
Copy Markdown
Contributor

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

Project Deployment Actions Updated (UTC)
relay-sdk-demo Ready Ready Preview, Comment Jul 24, 2026 6:53pm

Request Review

@github-actions

github-actions Bot commented Jul 24, 2026

Copy link
Copy Markdown
Contributor

Claude finished @pedromcunha's task in 21s —— View job


Code Review

Code Review -- No issues found.

Reviewed 2 files, 8 lines changed. Checked for correctness, security, error handling, and repo guidelines compliance.

The change adds Robinhood Wallet to WalletChainRestricted (its supported chain IDs) and registers robinhoodwallet → robinhood in NormalizedWalletName, following the exact pattern of the existing ronin/porto entries. Both keys are consistent (robinhood), the changeset is present and correctly scoped as a patch to @relayprotocol/relay-kit-ui, and formatting matches the file's conventions.

@github-actions

github-actions Bot commented Jul 24, 2026

Copy link
Copy Markdown
Contributor

Claude finished @pedromcunha's task in 43s —— View job


📝 Docs Drift Check — No docs impact

This PR does not touch surfaces that require a relayprotocol/relay-docs update.

@pedromcunha
pedromcunha merged commit d8c713c into main Jul 24, 2026
6 checks passed
@pedromcunha
pedromcunha deleted the pedro/int2-991-set-wallet-restrictions-on-robinhood-wallet branch July 24, 2026 18:55
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