daveroga/rationalize configs

[daveroga]

Fixes issue #355 .

The configuration for this application is becoming fragmented. It now exists in:

• Some constants.mjs files.
• Environment files
• config module files
• env-cmd module files

Probably it is sensible just to use one of these.

Steps to solve this:

• Used config module files for tests and removed constants.mjs.
• Removed env-cmd module files.
• Tried to use config module in cli but then found some problems with babel/webpack and the wallet. Some constants are cli specific and should be in the cli project.

[druiz0992]

I would like to keep web3 node url out of repo code. PR #334 proposes a way to do it. I suggest once it is approved, just reuse it

[daveroga]

PR already solved but e2e test failing due to issue #427 that should be solved first.

[daveroga]

ganache-test seems to fail sometimes. Forced modify a file to test it again.

[ChaitanyaKonda]

This seems to add 263 commits in total. Most of which are commits that already exist in the master.
Merging this has the risk of breaking the code and adding back logic that we removed.

Could you revert this to just the commits from this PR please?

[daveroga]

Wrong commits added after rebase were reverted and rebase again from master.

[signorecello]

LGTM but just mind some tests are being removed by #490

[-actions]

🎉 This PR is included in version 1.16.1 🎉

The release is available on:

• npm package (@latest dist-tag)
• release

Your semantic-release bot 📦🚀