MetaMask / MetaMask/core

Queued request controller to flush queue on network change for a domain

Open
#4,123 0 comments 0 reactions 0 assignees View on GitHub
team-wallet-api-platform
Dominant language
TypeScript
Stars
413
Forks
308
Avg merge
1d 4h
Merged PRs (30d)
253

Description

The selected network can change in other circumstances as well. For example if the user switches the network dropdown in the popup, according tot he designs it should also update the dapp-selected network (I know that's not implemented yet, but it will be soon).

So we may want to trigger this flushing somewhere else, which would capture both cases. e.g. when the state changes, or in a method responsible for making that state change

_Originally posted by @Gudahtt in https://github.com/MetaMask/core/pull/4064#discussion_r1526651126_

Contributor guide

No contributing guide indexed for this repository

Research direction

Start by locating the queued request controller and the state-change method that updates the selected network. Trace the existing network-change path and the popup network-dropdown path described in the issue, then identify a shared point for flushing the queue; done means both network-change cases consistently flush queued requests.

Written by the indexing model from the issue text.

Assessment

Tech stack
typescript
Domain
backend
Issue type
Feature
Difficulty
4/5
Estimated time
3-5 days
Activity status
Stale
Clarity
Needs clarification
Newbie friendliness
35/100

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.