multiple providers: work tracker issue

[frankiebee]

goal:
initialize all providers instead of creating and destroying on network switch while maintaining same external behavior

high level todos:

• (Migration needed) pass in all end points
• remove the storage of endpoints from preferences controller
• initialize all providers
• continue same external behavior of the provider
• QA
• some UI tweaks for sanity of params
• frankie dont go to overboard

[frankiebee]

current status: broke up provider initialization and proxy management into to files

notes/low level todos:
default networks:
- [ ] never initialize any local hosts unless it's tests. (side option try to initialize and have graceful fallback?)

• if selected network is a non infura fallback to infura if possible
  - [ ] network.stop()
  metamaskController:

• async updateAndSetCustomRpc (rpcUrl, chainId, ticker = 'ETH', nickname, rpcPrefs) {

• async setCustomRpc (rpcTarget, chainId, ticker = 'ETH', nickname = '', rpcPrefs = {}) {

• async delCustomRpc (rpcTarget) {

[frankiebee]

MIGRATION NEEDS:

• PreferencesController.frequentRpcListDetail -> NetworkController.networkConfigs

- [ ] network controller needs selected network and networkconfigs
include the defaults in networkConfigs
check what default ticker is needed

[frankiebee]

• if possible unify object in network controller:
• rpcTarget -> rpcUrl
• nickname -> name
• flatten rpcPrefs

 networkConfig = {
    rpcUrl,
    type, // default#type(chainId) || custom#type(chainId) ???
    name,
    chainId,
    blockExplorer,
  }

network class config:

  config = { // should ever only be things for the provider or network class no extra bs
    rpcUrl,
    type,
    chainId, // for ethClassic
  }

[frankiebee]

• ui needs some method updates
  more things to come

[frankiebee]

updateAndSetCustomRpc -> takes a single object
async updateAndSetCustomRpc (networkConfig) {

[frankiebee]

actual:

networkConfig = {
    rpcUrl,
    type, // infura#eth:mainnet || custom#eth:rpc
    custom: {
      name,
      chainId,
      blockExplorer,
      ticker,
    }
  }

[frankiebee]

• ensure the initialize param is discarded if passed to the constructor

[frankiebee]

wooooooooooo

  1104 passing (5s)
  2 failing

sooooo close and yet so far

[danfinlay]

Let's take a little time to explore H4v0c's work on abstracting multiple chain providers, see if there's a chance to leverage their work at all: http://chainstackjs.org/

[frankiebee]

• needs docs