[ENG-3496] fix: utxo 500 issue add a fallback option to our api httpsbtc

[teebszet]

🔘 PR Type

• Enhancement

📜 Background

Issue Link: ENG-3496
depends on: secretkeylabs/xverse-core#339

🔄 Changes

• added an (optional) input to change network screen for fallback btc api url. it can be saved with an empty input
• fixed the UX so that switching between mainnet/testnet will remember previously set urls
• renamed the Reset URL button to 'Reset to default'
• added a quick validation for BTC URL and Stacks URL inputs, to not allow empty
• uses core version [ENG-3496] fix: utxo 500 issue add a fallback option to our api httpsbtc xverse-core#339 where BitcoinEsploraApiProvider will retry on the fallbackUrl (if provided) on:
  • request timeouts
  • response 4xx 5xx codes
• some minor restyling on the change network screen
  • use the revamped feedback errors
  • use the revamped input 'X' button (cross in circle icon)

Impact:

• anywhere useBtcClient was used, will have a fallback url, which is configurable from the settings screen
  • can verify this by triggering calls to btcclient and checking the baseURL of the network requests
  • btcclient is used in sendBtc, sendOrdinal, sendRareSats, restoreOrdinal, sendBrc20, btcRbf, transaction history, sign psbt request, edit btc fees, one step brc20, create inscription
• note that BitcoinEsploraApiProvider needs to still be used without the fallback url, when validating each custom URL input (on the change network screen itself)

🖼 Screenshot / 📹 Video

Screen.Recording.2024-01-09.at.6.58.36.PM.mov

✅ Review checklist

Please ensure the following are true before merging:

• Code Style is consistent with the project guidelines.
• Code is readable and well-commented.
• No unnecessary or debugging code has been added.
• Security considerations have been taken into account.
• The change has been manually tested and works as expected.
• Breaking changes and their impacts have been considered and documented.
• Code does not introduce new technical debt or issues.

[victorkirov]

I left a thread of comments but maybe the idea is to not use the URL value from the config endpoint? If that's the case, then I'm happy to approve 👍

[teebszet]

I left a thread of comments but maybe the idea is to not use the URL value from the config endpoint? If that's the case, then I'm happy to approve 👍

I asked @m-aboelenein here: https://secretkeylabs.slack.com/archives/C05QCJM9D1N/p1704804634811279

and looks like we can remove that functionality

[victorkirov]

So much cleaner! 🎉

[DuskaT021]

@teebszet needs conflicts resolved

[teebszet]

@DuskaT021 done!

[github-actions]

Test with build here: https://github.com/secretkeylabs/xverse-web-extension/actions/runs/7475252053