chore: remove deprecated network selector from settings

[juanmigdr]

Description

The network selector in the settings view is very outdated and inconsistent with the new Bip-44 UI. This is causing:

• New bugs raised
• Double Developer effort when dealing with networks
• Inconsistencies

Therefore, after consulting with @Tlees-MMI and @alfeng6 we have come to the conclussion of fully removing it from the settings view. From now on if users want to do anything related to network management they can use the network selector from the Wallet Screen. Here is part of the discussion:

Changelog

CHANGELOG entry: removal of network selector in settings

Related issues

Fixes:

• https://consensyssoftware.atlassian.net/browse/ASSETS-1354 | [Bug]: Wrong network name when editing a network settings #20458
• https://consensyssoftware.atlassian.net/browse/ASSETS-1339 | [Bug]: Popular networks are categorized a Custom Networks in the settings #20362

Manual testing steps

Feature: my feature name

  Scenario: user [verb for user action]
    Given [describe expected initial app state]

    When user [verb for user action]
    Then [describe expected outcome]

Screenshots/Recordings

Before

Screen.Recording.2025-10-29.at.12.08.07.mov

After

Screen.Recording.2025-10-29.at.11.01.32.mov

Pre-merge author checklist

• I’ve followed MetaMask Contributor Docs and MetaMask Mobile Coding Standards.
• I've completed the PR template to the best of my ability
• I’ve included tests if applicable
• I’ve documented my code using JSDoc format if applicable
• I’ve applied the right labels on the PR (see labeling guidelines). Not required for external contributors.

Pre-merge reviewer checklist

• I've manually tested the PR (e.g. pull and build branch, run the app, test code being changed).
• I confirm that this PR addresses all acceptance criteria described in the ticket it closes and includes the necessary testing evidence such as recordings and or screenshots.

---

Note

Removes the deprecated Networks settings screen and entry, shifting network management to the wallet network switcher and updating navigation, unit tests, and e2e flows accordingly.

• Settings:
  • Remove Networks drawer item and related handler from Views/Settings.
• Navigation:
  • Remove NetworksSettings screen from MainNavigator.
• NetworksSettings Removal:
  • Delete Views/Settings/NetworksSettings component, unit tests, and snapshots.
• Tests/E2E:
  • Update Settings snapshot and unit tests to exclude networks-settings.
  • Remove NETWORKS selector and page object methods.
  • Refactor e2e flows to manage networks via wallet network switcher (WalletView/NetworkListModal) instead of Settings (e.g., add/delete networks, permission cleanup, localhost helper).

^{Written by Cursor Bugbot for commit d94fbf5. This will update automatically on new commits. Configure here.}

[sonarqubecloud]

Quality Gate passed

Issues
0 New issues
0 Accepted issues

Measures
0 Security Hotspots
0.0% Coverage on New Code
0.0% Duplication on New Code

See analysis details on SonarQube Cloud

[cursor]

Bug: Test Missing Navigation to NetworkView

The test "should add BSC network" calls NetworkView.tapAddNetworkButton() without first navigating to the NetworkView. The previous test "should go to settings then networks" which provided this navigation was removed, but this test was not updated accordingly. The test will fail because NetworkView is not visible. It should navigate to the network view first using await WalletView.tapNetworksButtonOnNavBar(); before attempting to interact with NetworkView elements, similar to how addLocalhostNetwork() was updated in viewHelper.ts.

e2e/specs/quarantine/deeplinks.failing.ts#L80-L89

metamask-mobile/e2e/specs/quarantine/deeplinks.failing.ts

Lines 80 to 89 in d94fbf5

	it('should add BSC network', async () => {
	// Tap on Add Network button
	await TestHelpers.delay(3000);
	await NetworkView.tapAddNetworkButton();
	await Assertions.expectElementToBeVisible(NetworkView.networkContainer);
	await NetworkView.tapNetworkByName(
	PopularNetworksList.BNB.providerConfig.nickname,
	);

 

---

[juanmigdr]

Screen has been removed so no deep-linking for it

[Prithpal-Sooriya]

Sounds good. Great job updating quarantined tests! Reduces confusion once this comes out of quarantine.