chore: remove WS endpoint requirement from settings

albert-llimos commented 2 weeks ago

Checklist

Please conduct a thorough self-review before opening the PR.

[ ] I am confident that the code works.
[ ] I have written sufficient tests.
[ ] I have written and tested required migrations.
[ ] I have updated documentation where appropriate.

Summary

We don't use the websocket endpoint for Solana so we want to remove the need for it in the settings.

codecov[bot] commented 2 weeks ago

Codecov Report

All modified and coverable lines are covered by tests :white_check_mark:

Project coverage is 70%. Comparing base (630706a) to head (1837cc8).

Additional details and impacted files

```diff @@ Coverage Diff @@ ## release/1.6 #5221 +/- ## =========================================== Coverage 70% 70% =========================================== Files 481 481 Lines 86252 86255 +3 Branches 86252 86255 +3 =========================================== + Hits 60416 60420 +4 Misses 22550 22550 + Partials 3286 3285 -1 ```

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.

dandanlen commented 2 weeks ago

I've squashed and moved this to #5220

chainflip-io / chainflip-backend