Update Wallet to use ENV file for Provider URL

[camsjams]

Per #203 there is a todo within @fuel-ts/wallet to remove hardcoded FUEL_NETWORK_URL value of 'http://127.0.0.1:4000/graphql'

[danielbate]

Would be happy to pick this up over the weekend if it's still available? There seems to be lots of hardcoded usage of 'http://127.0.0.1:4000/graphql' throughout.

[arboleya]

@danielbate If you ignore the occurrences inside test files (it seems to me like those should remain as is), only one is left:

fuels-ts/packages/wallet/src/constants.ts

Line 2 in 2d70903

export const FUEL_NETWORK_URL = 'http://127.0.0.1:4000/graphql';

It is used as a default param assignment in some methods, like this:

fuels-ts/packages/wallet/src/base-locked-wallet.ts

Line 42 in 7d3416d

constructor(publicKey: string | AbstractAddress, provider: string | Provider = FUEL_NETWORK_URL) {

So, deleting the FUEL_NETWORK_URL constant would require changing all relevant method signatures using it (i.e., removing the default param assignment) and updating all broken usages of such methods.

@FuelLabs/sdk-ts Makes sense?

[LuizAsFight]

makes sense @arboleya , maybe we could create a env var to hold http://127.0.0.1:4000/graphql instead ? then could do something like....

export const FUEL_NETWORK_URL = process.env.FUEL_PROVIDER_URL

[arboleya]

Closing as completed.