-
Notifications
You must be signed in to change notification settings - Fork 966
feat(wallet): iOS Wallet Network Settings #23437
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
Conversation
ios/brave-ios/Sources/BraveWallet/Crypto/Stores/NetworkStore.swift
Outdated
Show resolved
Hide resolved
ios/brave-ios/Sources/BraveWallet/Crypto/Stores/NetworkStore.swift
Outdated
Show resolved
Hide resolved
ios/brave-ios/Sources/BraveWallet/Settings/NetworkListView.swift
Outdated
Show resolved
Hide resolved
ios/brave-ios/Tests/BraveWalletTests/NetworkSelectionStoreTests.swift
Outdated
Show resolved
Hide resolved
ios/brave-ios/Sources/BraveWallet/Crypto/Stores/NetworkStore.swift
Outdated
Show resolved
Hide resolved
ios/brave-ios/Sources/BraveWallet/Settings/NetworkListView.swift
Outdated
Show resolved
Hide resolved
3bd3459
to
c152ce1
Compare
[puLL-Merge] - brave/brave-core@23437 Here is my review of the PR: DescriptionThis PR makes several changes related to how network visibility is handled in the Brave Wallet. The main changes are:
The motivation seems to be to improve the UX around network selection and visibility and to avoid issues with preferences getting out of sync with core. Security HotspotsThis change does not appear to introduce any new security vulnerabilities. The main sensitive data being handled are the user's crypto assets, but the changes here are focused on the UI layer and do not fundamentally alter how assets are stored or accessed. ChangesChangesChanges by file: CryptoTabsView.swift
FiltersDisplaySettingsView.swift
NetworkFilterView.swift
NetworkPicker.swift
NetworkSelectionRootView.swift
PortfolioView.swift
AssetSearchView.swift
AccountsStore.swift
NetworkStore.swift
Web3SettingsView.swift
WalletPreferences.swift
RpcServiceExtensions.swift
SuggestedNetworkView.swift
In summary, this is a significant refactor of how network visibility is handled, but the changes are fairly contained to the network and asset management layers. Thorough testing of network-related features would be prudent given the scope of changes. |
Resolves brave/brave-browser#37741
The
Show Test Networks
toggle is now removedAll networks including primary networks, sub-networks, test networks, custom neworks, are all listed in this screen
Submitter Checklist:
QA/Yes
orQA/No
;release-notes/include
orrelease-notes/exclude
;OS/...
) to the associated issuenpm run test -- brave_browser_tests
,npm run test -- brave_unit_tests
wikinpm run presubmit
wiki,npm run gn_check
,npm run tslint
git rebase master
(if needed)Reviewer Checklist:
gn
After-merge Checklist:
changes has landed on
Test Plan:
Show Test Networks
is turned offeye-closed
icon indicates they are hiddenShow Test Networks
. You should see test networks for each coin type are listed witheye-opened
icon indicates they are not hidden.eye-opened
icon but grey-out indicates hiding this network is disabled. This also means this is the default network for this coin type.Edit
is availableEdit
/Delete
/Set as Default
eye-opened
iconeye-closed
icon if its hiddeneye-closed
iconeye-opened
icon if its not hidden