Matt Luongo
Matt Luongo
That really depends on how your `web3` is injected, as far as I know? See [here](https://github.com/keep-network/tbtc-dapp/blob/167115d9d0341db25c28513081d4628a213e2d05/src/wrappers/web3.js#L43). I think supporting web3 generically before explicit multi-wallet support makes sense- otherwise we're excluding...
I'm struggling to understand that alternative behavior you'd like to see, before we roll out multi-wallet support.
> When I'm setting up a send ERC20 transaction in MM it shows me the token recipient on my Ledger UI. So I guess that could be confusing. I wonder...
I'd rather change both together to keep them as similar as possible... in an ideal world this screen shows the same thing as the device.
Ooh, I like this — @VladUXUI what do you think, is there risk of confusion here?
For background: The original services idea was partially inspired by [`component`](https://github.com/stuartsierra/component), a declarative Clojure DI library. It's all about smaller components to isolate side-effects, eg using a "service" to wrap...
> What is the best way to break the dependency cycle? Service1 Service2 method calls. > Should we use the emitter to communicate backward to the original service? Yes. The...
> Hmmm… If I'm not mistaken, the same address on RSK is not controlled by the same keys, as far as I understand it. This can be true on account...
Thanks for the contribution @abhiram11! Any chance you can merge `man` into your branch? @Shadowfiend can you take a peek at this when you get a minute?
@banteg we're now getting consistent 404s with this list — it's been a few days