[WALL] Rostislav / WALL-3069 / Use requests instead of subscriptions for exchange rates in transfer messages and transfer inputs

[rostislav-deriv]

Changes:

• add useGetExchangeRate
• switch to using timer-initiated exchange rates requests instead of subscriptions in transfer

Screenshots:

Screen.Recording.2023-12-13.at.13.44.51.mov

[vercel]

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name	Status	Preview	Updated (UTC)
deriv-app	✅ Ready (Inspect)	Visit Preview	Dec 15, 2023 3:51pm

[github-actions]

A production App ID was automatically generated for this PR. (log)

• PR: [WALL] Rostislav / WALL-3069 / Use requests instead of subscriptions for exchange rates in transfer messages and transfer inputs #12229
• URLs:
  • w/ App ID + Server: https://deriv-app-git-fork-rostislav-deriv-rostislav-transfer-me-b0527e.binary.sx?qa_server=red.derivws.com&app_id=31385
  • Original: https://deriv-app-git-fork-rostislav-deriv-rostislav-transfer-me-b0527e.binary.sx
• App ID: 31385

Click here to copy & paste above information.

- **PR**: [https://github.com/binary-com/deriv-app/pull/12229](https://github.com/binary-com/deriv-app/pull/12229)
- **URLs**:
    - **w/ App ID + Server**: https://deriv-app-git-fork-rostislav-deriv-rostislav-transfer-me-b0527e.binary.sx?qa_server=red.derivws.com&app_id=31385
    - **Original**: https://deriv-app-git-fork-rostislav-deriv-rostislav-transfer-me-b0527e.binary.sx
- **App ID**: `31385`

[github-actions]

🚨 Lighthouse report for the changes in this PR:

Category	Score
🔺 Performance	16
🟧 Accessibility	76
🟢 Best practices	92
🟢 SEO	92
🟧 PWA	80

Lighthouse ran with https://deriv-app-git-fork-rostislav-deriv-rostislav-transfer-me-b0527e.binary.sx/

[coveralls]

coverage: 32.613% (+0.01%) from 32.602%
when pulling 8b0151d on rostislav-deriv:rostislav/transfer-messages-requests-for-exchange-rates
into c7ab037 on binary-com:master.

[ghost]

whats the reason for that switch? optimise / backend complaints about too many subscriptions etc?

[nijil-deriv]

whats the reason for that switch? optimise / backend complaints about too many subscriptions etc?

We only require the value to be updated every 60s as per requirement. If we subscribe to exchange_rate, we cannot guarantee that.

[rostislav-deriv]

Yup, this PR makes the exchange rates updates to input values and transfer messages synced and initiated by the timer.

[ghost]

worth adding tests? sooner rather than later tests for all new stuff are gonna be expected to be covered in unit or component tests

[sonarqubecloud]

Quality Gate passed

Kudos, no new issues were introduced!

0 New issues
0 Security Hotspots
No data about Coverage
1.2% Duplication on New Code

See analysis details on SonarCloud

[github-actions]

❌ Smoke test run (1) failed. See logs for details: Visit Action

[github-actions]

❌ Smoke test run (1) failed. See logs for details: Visit Action

[github-actions]

❌ Smoke test run (2) failed. See logs for details: Visit Action

[rostislav-deriv]

Tests for useTransferMessages will be added in a separate soon-to-be-created-by-me PR
cc @wojciech-deriv