Skip to content

Commit 5732b4f

Browse files
authored
fix: hard code remove GNS feature flag (#22961)
## **Description** Hardcode `isRemoveGlobalNetworkSelectorEnabled` feature flag ## **Changelog** CHANGELOG entry:null ## **Related issues** Fixes: ## **Manual testing steps** ```gherkin 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** `~` ### **After** `~` ## **Pre-merge author checklist** - [x] I’ve followed [MetaMask Contributor Docs](https://github.com/MetaMask/contributor-docs) and [MetaMask Mobile Coding Standards](https://github.com/MetaMask/metamask-mobile/blob/main/.github/guidelines/CODING_GUIDELINES.md). - [x] I've completed the PR template to the best of my ability - [x] I’ve included tests if applicable - [x] I’ve documented my code using [JSDoc](https://jsdoc.app/) format if applicable - [x] I’ve applied the right labels on the PR (see [labeling guidelines](https://github.com/MetaMask/metamask-mobile/blob/main/.github/guidelines/LABELING_GUIDELINES.md)). Not required for external contributors. ## **Pre-merge reviewer checklist** - [x] I've manually tested the PR (e.g. pull and build branch, run the app, test code being changed). - [x] 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. <!-- CURSOR_SUMMARY --> --- > [!NOTE] > Hardcodes `isRemoveGlobalNetworkSelectorEnabled` to always return `true`, removing the environment flag check. > > <sup>Written by [Cursor Bugbot](https://cursor.com/dashboard?tab=bugbot) for commit 229cac5. This will update automatically on new commits. Configure [here](https://cursor.com/dashboard?tab=bugbot).</sup> <!-- /CURSOR_SUMMARY -->
1 parent 522d6b6 commit 5732b4f

File tree

1 file changed

+1
-2
lines changed

1 file changed

+1
-2
lines changed

app/util/networks/index.js

Lines changed: 1 addition & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -672,8 +672,7 @@ export const getIsNetworkOnboarded = (chainId, networkOnboardedState) =>
672672
export const isPermissionsSettingsV1Enabled =
673673
process.env.MM_PERMISSIONS_SETTINGS_V1_ENABLED === 'true';
674674

675-
export const isRemoveGlobalNetworkSelectorEnabled = () =>
676-
process.env.MM_REMOVE_GLOBAL_NETWORK_SELECTOR === 'true';
675+
export const isRemoveGlobalNetworkSelectorEnabled = () => true;
677676

678677
// The whitelisted network names for the given chain IDs to prevent showing warnings on Network Settings.
679678
export const WHILELIST_NETWORK_NAME = {

0 commit comments

Comments
 (0)