ACINQ / phoenix

Phoenix is a self-custodial Bitcoin wallet using Lightning to send/receive payments.
https://phoenix.acinq.co
Apache License 2.0
632 stars 95 forks source link

Remove most of wallet-context usage #406

Closed dpad85 closed 11 months ago

dpad85 commented 11 months ago

This PR simplifies the WalletContext object as well as the logic fetching this resource from the acinq endpoint. The idea behind that PR is that the data in this wallet context are not critical, and failing to access this resource should not block the app (see #400).

Instead, this context only provides information that improve user experience, such as the warning regarding the mempool. These information are not vital to the application. Note that most of the data in the json are legacy and were already unused.

As a side-effect, the wallet params can now be hardcoded, including the trampoline fees settings.

@robbiehanson This affects the iOS app, since several references have been renamed/removed. The iOS app will have to be fixed.