-
Notifications
You must be signed in to change notification settings - Fork 609
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
network expansion #6149
network expansion #6149
Conversation
@@ -22,26 +23,27 @@ import { useSwapContext } from '../providers/swap-provider'; | |||
import { BLANK_BASE64_PIXEL } from '@/components/DappBrowser/constants'; | |||
|
|||
const networkBadges = { |
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.
alphabetized these.
New and removed dependencies detected. Learn more about Socket for GitHub ↗︎
🚮 Removed packages: npm/@rainbow-me/[email protected] |
@@ -102,16 +102,17 @@ export const DEFAULT_CONFIG: RainbowConfig = { | |||
data_endpoint: DATA_ENDPOINT || 'wss://api-v4.zerion.io', | |||
data_origin: DATA_ORIGIN, |
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.
alphabetizes these.
@@ -199,6 +199,7 @@ const getColorsByTheme = (darkMode?: boolean) => { | |||
[ChainId.avalanche]: '#E84142', | |||
[ChainId.degen]: '#A36EFD', | |||
[ChainId.blast]: '#25292E', | |||
[ChainId.apechain]: '#0054FA', |
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.
ok i missed this one on my PRs, is it possible to use native assets color? or maybe we're missing this color from the backend data
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.
Looked into this, they're different from native asset colors so I dunno. @christianbaroni any idea on this?
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.
lgtm
maybe we could add chain images and colors from backend too? I didn't add them in my PRs
For sure! I think both would be a great addition to the network expansion work. iirc we haven't done images yet because of how shadows work in the app but I may be wrong (cc @christianbaroni) |
More info https://rainbowhaus.slack.com/archives/C0468CDBE75/p1727380431942769?thread_ts=1727377244.944789&cid=C0468CDBE75
Fixes APP-1886