-
Notifications
You must be signed in to change notification settings - Fork 14
feat: Solana + Tron withdrawals + chain-expansion polish #2398
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
Merged
Merged
Changes from all commits
Commits
Show all changes
25 commits
Select commit
Hold shift + click to select a range
429279b
Merge pull request #2391 from peanutprotocol/dev
jjramirezn 82cffcc
fix: migrate root validator before mixed spends sign the Rain admin sig
Hugo0 d85278e
fix: same migration gate for the sign-only spend path + overlay the m…
Hugo0 bfe995e
chore: clear lint deltas — barrel import, orphaned decimals import, c…
Hugo0 e07d73c
refactor: one shared spend preflight for both engines — drift here is…
Hugo0 7985712
fix: harden the migration gate per adversarial review — 7 verified fi…
Hugo0 c8c3da3
chore: clear the lint annotations visible on the PR diff
Hugo0 f03ffd2
Merge pull request #2392 from peanutprotocol/hotfix/migrate-before-mi…
jjramirezn 2e27f52
fix: remove unauthenticated public Discord relay endpoint
jjramirezn c4cb821
Merge pull request #2395 from peanutprotocol/fix/remove-unauth-discor…
Hugo0 14ebb1f
feat: sync Rhino chain support with live catalogs
Hugo0 bde3487
feat: wire new Rhino chains into the withdraw selector gate
Hugo0 58aa9d9
fix: search withdraw tokens across the Rhino destination set
Hugo0 60e8184
feat: add Kaia + Plasma deposit chains
Hugo0 dc29b19
Merge pull request #2396 from peanutprotocol/hotfix/rhino-chain-catal…
Hugo0 7de82b9
Merge remote-tracking branch 'origin/main' into merge/main-into-dev-2…
Hugo0 bbadbf1
feat: Solana + Tron withdrawals, plus chain-expansion polish
Hugo0 83ed62a
feat: per-chain rollout flags (PostHog) for one-by-one chain launches
Hugo0 f5e2156
refactor: general useFeatureFlag primitive + useChainRollout as thin …
Hugo0 ea63b6c
fix: final-review FE corrections — context-level non-EVM merge, react…
Hugo0 0ed73e9
refactor: CHAIN_REGISTRY — one source of truth for every FE chain fact
Hugo0 a973865
feat: enable Base withdrawals (verified) + fully consolidate derivati…
Hugo0 5c7ec1d
fix: withdraw receipts always link the source-chain explorer (CodeRab…
Hugo0 89e0621
test: leak tripwire — non-EVM synthetic records stay out of non-withd…
Hugo0 bd5af5f
Merge pull request #2401 from peanutprotocol/refactor/chain-registry
Hugo0 File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
There are no files selected for viewing
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file was deleted.
Oops, something went wrong.
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -0,0 +1,24 @@ | ||
| import ChainChip from './ChainChip' | ||
| import { SUPPORTED_EVM_CHAINS, CHAIN_LOGOS, EVM_DEPOSIT_TOKEN_EXCEPTIONS } from '@/constants/rhino.consts' | ||
| import { useChainRollout } from '@/hooks/useChainRollout' | ||
|
|
||
| /** | ||
| * The rollout-gated EVM deposit chain chips, annotated with per-chain token | ||
| * exceptions (USDT-only chains) — a USDC deposit on a chain where Rhino only | ||
| * accepts USDT has no webhook, so the annotation is a funds-safety surface, | ||
| * not decoration. Shared by ChooseNetworkDrawer and SupportedNetworksModal. | ||
| */ | ||
| const EvmChainChips = () => { | ||
| const isChainRolledOut = useChainRollout() | ||
| return ( | ||
| <> | ||
| {SUPPORTED_EVM_CHAINS.filter(isChainRolledOut).map((chain) => { | ||
| const tokenException = EVM_DEPOSIT_TOKEN_EXCEPTIONS[chain] | ||
| const label = tokenException ? `${chain} · ${tokenException.join('/')} only` : chain | ||
| return <ChainChip key={chain} chainName={label} chainSymbol={CHAIN_LOGOS[chain]} /> | ||
| })} | ||
| </> | ||
| ) | ||
| } | ||
|
|
||
| export default EvmChainChips |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Oops, something went wrong.
Oops, something went wrong.
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Uh oh!
There was an error while loading. Please reload this page.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
📐 Maintainability & Code Quality | 🟠 Major | ⚡ Quick win
Fix the test setup’s lint errors.
require()at Lines 509 and 533 violates@typescript-eslint/no-require-imports, whilemockSetLoadingStateat Line 521 is unused. ImportloadingStateContextstatically, remove the localrequire()bindings, and either remove the mock or pass it as the provider’ssetLoadingStatevalue.Proposed cleanup
Also applies to: 521-522, 533-533
🧰 Tools
🪛 ESLint
[error] 509-509: A
require()style import is forbidden.(
@typescript-eslint/no-require-imports)🤖 Prompt for AI Agents
Source: Linters/SAST tools
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Not in this PR's scope — qr-pay-states.test.tsx isn't touched by this branch (this review ran before back-merge #2397 landed on dev; the current diff vs dev has zero hits on this file).
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
🧩 Analysis chain
🏁 Script executed:
Length of output: 3646
🏁 Script executed:
Length of output: 690
@Hugo0, thanks for the context. Just to double check, I re-ran a check against the current state of the PR and the file still shows up in the diff:This may just mean the branch was updated after your comment (e.g., a rebase/merge bringing dev's version of the file back into the diff), so the
require()lines may now be back in scope. If that's the case and you'd still like to defer the lint cleanup to a separate PR, that's totally fine — just let me know and I'll leave it alone. Otherwise, happy to help with the fix here if it's now in scope.🧬 Code Graph Analysis Results
Relevant snippet —
src/types/capabilities.ts(lines 63-90)Relevant snippet —
src/app/(mobile-ui)/qr-pay/page.tsx(lines 101-260)