1
0
mirror of https://github.com/kremalicious/metamask-extension.git synced 2024-10-22 11:22:43 +02:00

Fixes to the Linea Goerli implementation (#18290)

* Ensure that NonInfuraDefaultNetworks are only selected in the dropdown if they are the currently selected network

* Ensure Linea Goerli network appears in network settings tab if added manually
This commit is contained in:
Dan J Miller 2023-03-23 09:07:28 -07:00 committed by Dan Miller
parent 98ed05c7c0
commit 1dc09c027c
2 changed files with 27 additions and 30 deletions

View File

@ -280,22 +280,19 @@ class NetworkDropdown extends Component {
}
renderNonInfuraDefaultNetwork(network) {
const {
provider: { type: providerType },
setRpcTarget,
} = this.props;
const { provider, setRpcTarget } = this.props;
const isCurrentRpcTarget = providerType === NETWORK_TYPES.RPC;
const { chainId, ticker, blockExplorerUrl } = BUILT_IN_NETWORKS[network];
const networkName = NETWORK_TO_NAME_MAP[network];
const rpcUrl = CHAIN_ID_TO_RPC_URL_MAP[chainId];
const isCurrentRpcTarget =
provider.type === NETWORK_TYPES.RPC && rpcUrl === provider.rpcUrl;
return (
<DropdownMenuItem
key={network}
closeMenu={this.props.hideNetworkDropdown}
onClick={async () => {
const { chainId, ticker, blockExplorerUrl } =
BUILT_IN_NETWORKS[network];
const networkName = NETWORK_TO_NAME_MAP[network];
const rpcUrl = CHAIN_ID_TO_RPC_URL_MAP[chainId];
await setRpcTarget(rpcUrl, chainId, ticker, networkName, {
blockExplorerUrl,
});
@ -317,7 +314,7 @@ class NetworkDropdown extends Component {
data-testid={`${network}-network-item`}
style={{
color:
providerType === network
provider.type === network
? 'var(--color-text-default)'
: 'var(--color-text-alternative)',
}}

View File

@ -30,11 +30,13 @@ import NetworksTabContent from './networks-tab-content';
import NetworksForm from './networks-form';
import NetworksFormSubheader from './networks-tab-subheader';
const defaultNetworks = defaultNetworksData.map((network) => ({
...network,
viewOnly: true,
isATestNetwork: TEST_CHAINS.includes(network.chainId),
}));
const defaultNetworks = defaultNetworksData
.map((network) => ({
...network,
viewOnly: true,
isATestNetwork: TEST_CHAINS.includes(network.chainId),
}))
.filter((network) => network.chainId !== CHAIN_IDS.LINEA_TESTNET);
const NetworksTab = ({ addNewNetwork }) => {
const t = useI18nContext();
@ -53,20 +55,18 @@ const NetworksTab = ({ addNewNetwork }) => {
const provider = useSelector(getProvider);
const networksTabSelectedRpcUrl = useSelector(getNetworksTabSelectedRpcUrl);
const frequentRpcNetworkListDetails = frequentRpcListDetail
.map((rpc) => {
return {
label: rpc.nickname,
iconColor: 'var(--color-icon-alternative)',
providerType: NETWORK_TYPES.RPC,
rpcUrl: rpc.rpcUrl,
chainId: rpc.chainId,
ticker: rpc.ticker,
blockExplorerUrl: rpc.rpcPrefs?.blockExplorerUrl || '',
isATestNetwork: TEST_CHAINS.includes(rpc.chainId),
};
})
.filter((network) => network.chainId !== CHAIN_IDS.LINEA_TESTNET);
const frequentRpcNetworkListDetails = frequentRpcListDetail.map((rpc) => {
return {
label: rpc.nickname,
iconColor: 'var(--color-icon-alternative)',
providerType: NETWORK_TYPES.RPC,
rpcUrl: rpc.rpcUrl,
chainId: rpc.chainId,
ticker: rpc.ticker,
blockExplorerUrl: rpc.rpcPrefs?.blockExplorerUrl || '',
isATestNetwork: TEST_CHAINS.includes(rpc.chainId),
};
});
let networksToRender = [...defaultNetworks, ...frequentRpcNetworkListDetails];