Skip to content

Refactor deposit address transaction tracking to use requests API#928

Merged
ted-palmer merged 1 commit intomainfrom
ted/fe-8323-update-onramp-ux-to-use-requests-api
Feb 10, 2026
Merged

Refactor deposit address transaction tracking to use requests API#928
ted-palmer merged 1 commit intomainfrom
ted/fe-8323-update-onramp-ux-to-use-requests-api

Conversation

@ted-palmer
Copy link
Collaborator

Switch OnrampModal and DepositAddressModalRenderer from polling /intents/status/v2 (via useExecutionStatus) to polling /requests/v2 (via new useDepositAddressStatus hook) using the deposit address as the user param.

The new hook wraps useRequests and maps the response to match the existing executionStatus interface, minimizing changes to consuming components.

Switch OnrampModal and DepositAddressModalRenderer from polling
/intents/status/v2 (via useExecutionStatus) to polling /requests/v2
(via new useDepositAddressStatus hook) using the deposit address as
the user param.

The new hook wraps useRequests and maps the response to match the
existing executionStatus interface, minimizing changes to consuming
components.

Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
@vercel
Copy link
Contributor

vercel bot commented Feb 6, 2026

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Actions Updated (UTC)
relay-sdk-demo Ready Ready Preview, Comment Feb 6, 2026 3:37pm

Request Review

@ted-palmer ted-palmer merged commit 1a1d6ce into main Feb 10, 2026
3 checks passed
@ted-palmer ted-palmer deleted the ted/fe-8323-update-onramp-ux-to-use-requests-api branch February 10, 2026 17:36
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants