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.
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
feat: Multichain
AssetList
#28386feat: Multichain
AssetList
#28386Changes from 160 commits
6ec980b
3b6d223
d670862
bf5913c
85d5161
72ea2cf
8b52780
4349f4f
8b83df4
bc82cb4
42ad7d4
c4ce6c7
5be0713
023b0ca
c9a2d8d
ae96cea
5b40705
f7f09da
85fe12d
0f3cd64
bdf921b
9bc4ecf
c6a408d
9b1529b
c8bcc0b
d1a3782
05fd6ef
9dfdfa8
04ea3d6
cf66072
6934434
15e4e47
a2074cd
6e80b34
836a11b
441d4cf
d18d25a
dd0b3f5
1e461ee
a51fec4
0d637fb
c923dfe
6a394a2
785de28
f461767
f1da818
878f407
e8a9d6a
245f8d3
1afebee
e99e2ef
ca590ec
d2191ac
70c0db4
6b7df4d
4e816c2
aaef377
48bff46
0450b04
fdc04ad
3ec3574
dcc8a64
0e7a872
ea95aca
3bf7b78
bb1f278
14dc84b
ef3ef10
ab6e7cc
44861d9
0b543e0
411fc1c
c2b6afe
69404a5
bb1d6bf
f6e56b3
8e752a1
6fac24b
12570a5
39a1285
95ce7de
f3a679b
b0c07c1
03e047d
3612b58
776c63b
9482953
bd035f3
23747ed
14c9b57
c308609
3c9049c
f81cbbc
93cd913
f290926
919faf5
1710bc7
ffe9e5f
46bc93c
5f0e632
f48e734
25c8a90
06112af
56470ad
42ce9be
6de2f95
ecb8508
a241f5e
9509d36
b5a0642
41f19f2
ffa995b
a58da57
65e2c18
3edf269
2dc23d3
d778834
6537cb6
ae73a3b
6397e30
74f2b8e
82d4c4e
3cc8773
f73c8f4
eea519f
ac3c389
3c2bc6b
21bd89a
3ff6b40
e55852e
929f406
5d06829
39ec4e7
1d747e6
9fe2f23
2691cb7
01c51de
e82bb27
f1077cf
9b4b058
9defc73
ed8cd61
994b363
c5ddc71
d5e56ec
776cecf
6a663a8
7e2b157
5d07e03
a7e2a77
156482a
cd49ad1
b4e4ba7
d49824f
5c3d4d3
031e8e3
40b36f5
b24e25a
18aa7e8
e127f07
7307904
b5da71e
e939574
c71bee0
f9fa8da
1a1737a
41adb56
aa3063f
57f8af1
01652f5
02dba24
bc5f345
0055289
c006bb6
25c27ea
ce6f520
6ceb087
dcf39fd
0ffa5d0
89b9ffc
8739893
572f998
82e1ddd
5ebc8e4
5b684ad
21aa0c4
11c51aa
bc2f132
f474448
a29b211
71c8879
4947aba
5fa2984
fc0722c
bf61b33
04c3e3f
ae00656
d18e085
43b0cfe
c147b3a
ae01bfc
9252c18
35f6287
8e57813
e0b49ed
0a72a62
0bee071
2df22cb
5c4c9b4
0e2c013
8bf9eaf
1df49df
bc459c2
1fff579
5980212
758fa6a
3b42f0c
2338fcc
a8364ff
4f70ccd
da79b45
836c2f5
704fd6f
85546a0
3273dba
1bdaf8d
817c9ba
ce01c11
e55b50d
a9b16e2
675780e
614b5ff
0bbaeb5
2c458c5
6e92ee4
e6b33f5
838e4a1
00c07aa
5b7ea11
0d77e0b
d4880c7
f0d722f
035fb5c
0867b5c
7f29e11
a610400
650aec5
b34ce81
006f12c
4d3aeba
File filter
Filter by extension
Conversations
Jump to
There are no files selected for viewing
Some generated files are not rendered by default. Learn more about how customized files appear on GitHub.
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.
Is this still true?
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.
In a perfect world, I think we should have a migration that runs and updates everyone's default network filter to all their
networkConfigurationByChainId
, rather than updating them on the fly in the UI.I'm not sure if it's totally necessary to be feature complete however. I can remove this comment if it's confusing.
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.
Should this use
ButtonIcon
?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.
Yeah we probably should. I'll make this change ty for pointing this out.