chore(ramp): upgrade ramp-sdk to @2.0.4

[georgeweiler]

Description

This upgrades @consensys/on-ramp-sdk to 2.0.3, which is a major version change that updates the getQuotes and getSellQuotes interfaces.

Related issues

Fixes:

Manual testing steps

1. Go to ramp experience and confirm that quotes and sell quotes are loaded.

Screenshots/Recordings

Before

After

Pre-merge author checklist

• I’ve followed MetaMask Contributor Docs and MetaMask Mobile Coding Standards.
• I've completed the PR template to the best of my ability
• I’ve included tests if applicable
• I’ve documented my code using JSDoc format if applicable
• I’ve applied the right labels on the PR (see labeling guidelines). Not required for external contributors.

Pre-merge reviewer checklist

• I've manually tested the PR (e.g. pull and build branch, run the app, test code being changed).
• I confirm that this PR addresses all acceptance criteria described in the ticket it closes and includes the necessary testing evidence such as recordings and or screenshots.

[github-actions]

CLA Signature Action: All authors have signed the CLA. You may need to manually re-run the blocking PR check if it doesn't pass in a few minutes.

[socket-security]

Updated dependencies detected. Learn more about Socket for GitHub ↗︎

Package	New capabilities	Transitives	Size	Publisher
npm/@consensys/[email protected] 🔁 npm/@consensys/[email protected]	None	0	418 kB	consensys-npm

View full report↗︎

[github-actions]

Bitrise

❌❌❌ pr_smoke_e2e_pipeline failed on Bitrise! ❌❌❌

Commit hash: 9f9ede1
Build link: https://app.bitrise.io/app/be69d4368ee7e86d/pipelines/47a9431e-ba13-4869-8bc5-93739718ca91

Note

• You can kick off another pr_smoke_e2e_pipeline on Bitrise by removing and re-applying the Run Smoke E2E label on the pull request

Tip

• Check the documentation if you have any doubts on how to understand the failure on bitrise

[wachunei]

LGTM

[NicolasMassart]

LGTM, but I would love a unit test here.

[NicolasMassart]

S: could you take this opportunity to add a unit test for the hook?

[wachunei]

We have it coming in a next PR, the hook will change a little bit so it is not worth it now, here it is: https://github.com/MetaMask/metamask-mobile/pull/13257/files#diff-66e86df8336a31e6a8432fb36644448548574f07052e57c265b4011145d0a423

[wachunei]

Quickly after adoption of v2.0.3 it was deprecated, this must be updated to v2.0.4

[github-actions]

Bitrise

❌❌❌ pr_smoke_e2e_pipeline failed on Bitrise! ❌❌❌

Commit hash: e22098f
Build link: https://app.bitrise.io/app/be69d4368ee7e86d/pipelines/d2e53cbc-c273-4aa9-abc6-7f54406cc70e

Note

• You can kick off another pr_smoke_e2e_pipeline on Bitrise by removing and re-applying the Run Smoke E2E label on the pull request

Tip

• Check the documentation if you have any doubts on how to understand the failure on bitrise

[github-actions]

Bitrise

❌❌❌ pr_smoke_e2e_pipeline failed on Bitrise! ❌❌❌

Commit hash: a1b66c5
Build link: https://app.bitrise.io/app/be69d4368ee7e86d/pipelines/15b4dd31-6182-4f35-a139-b1dc4cd8a300

Note

• You can kick off another pr_smoke_e2e_pipeline on Bitrise by removing and re-applying the Run Smoke E2E label on the pull request

Tip

• Check the documentation if you have any doubts on how to understand the failure on bitrise

[github-actions]

Bitrise

❌❌❌ pr_smoke_e2e_pipeline failed on Bitrise! ❌❌❌

Commit hash: 79f4b5d
Build link: https://app.bitrise.io/app/be69d4368ee7e86d/pipelines/d00569dc-17c9-451c-9166-b0338f15e536

Note

• You can kick off another pr_smoke_e2e_pipeline on Bitrise by removing and re-applying the Run Smoke E2E label on the pull request

Tip

• Check the documentation if you have any doubts on how to understand the failure on bitrise

[sonarqubecloud]

Quality Gate passed

Issues
0 New issues
0 Accepted issues

Measures
0 Security Hotspots
0.0% Coverage on New Code
0.0% Duplication on New Code

See analysis details on SonarQube Cloud

[NicolasMassart]

Looks good to me.
Thanks for the effort you made to provide the followup test PR! Very appreciated!