80 derivation path localstorage

[0xjocke]

Create a new derivationPathChange action.

• This action will set signer
• Update localstorage
• And finally, update redux store

Also set the initial state of the derivation path based on localstorage.
This will still will only be used for ledger wallets thanks to the getSignerConfig in snxJSConnector.js

Also a tiny refactor of setting the theme, see commit msg.
#80

[0xclem]

@bachstatter just reverted this one.

I think localStorage.getItem('derivationPath') in index.js might break Trezor. We need to make sure to set derivationPath to null when this wallet is selected.

[0xjocke]

Weird I thought it would be fine since getSignerConfig returns an empty object for Trezor.

const getSignerConfig = ({ type, networkId, derivationPath }) => {
	if (type === 'Ledger') {
		const DEFAULT_LEDGER_DERIVATION_PATH = "44'/60'/0'/";
		return { derivationPath: derivationPath || DEFAULT_LEDGER_DERIVATION_PATH };
	}
	if (type === 'Coinbase') {
		return {
			appName: 'Mintr',
			appLogoUrl: `${window.location.origin}/images/mintr-leaf-logo.png`,
			jsonRpcUrl: INFURA_JSON_RPC_URLS[networkId],
			networkId,
		};
	}

	return {};
};

[0xclem]

Yes of course. You're right, it should be fine.

So the only remaining issue is setSigner({ type, networkId }) in connectToWallet().
It doesn't set Ledger with the correct derivation path. It should check from either the store or localstorage is a value exists, and use it.