Skip to content

fix(checkout-widgets): Fetch pending withdrawals from the bridge contract#2680

Merged
keithbro-imx merged 18 commits into
mainfrom
pending-withdrawals-poc
Aug 19, 2025
Merged

fix(checkout-widgets): Fetch pending withdrawals from the bridge contract#2680
keithbro-imx merged 18 commits into
mainfrom
pending-withdrawals-poc

Conversation

@keithbro-imx

@keithbro-imx keithbro-imx commented Aug 15, 2025

Copy link
Copy Markdown
Contributor

Hi👋, please ensure the PR title follows the below standards:

  • PR is titled with conventional commit style naming: type(scope): message. For example: feat(passport): my new feature
  • If you have introduced modification that necessitates immediate adjustments by this SDK's users to their applications, clients, or integrations to avert disruptions to existing features or functionalities, add a ! after the type(scope), for example feat(passport)!: my new breaking feature

Summary

Replaces calls to the Checkout API with a direct calls to the RPC.

Detail and impact of the change

Added

Changed

Deprecated

Removed

Fixed

Security

Anything else worth calling out?

@nx-cloud

nx-cloud Bot commented Aug 15, 2025

Copy link
Copy Markdown

View your CI Pipeline Execution ↗ for commit d3d4049

Command Status Duration Result
nx affected -t build,lint,test ✅ Succeeded 2m 3s View ↗
nx run-many -p @imtbl/sdk,@imtbl/checkout-widge... ✅ Succeeded 1m 2s View ↗

☁️ Nx Cloud last updated this comment at 2025-08-18 07:32:14 UTC

@keithbro-imx keithbro-imx changed the title Pending withdrawals poc fix(checkout-widgets): Fetch pending withdrawals from the bridge contract Aug 18, 2025
@keithbro-imx keithbro-imx marked this pull request as ready for review August 18, 2025 06:36
@keithbro-imx keithbro-imx requested review from a team as code owners August 18, 2025 06:36
const pendingWithdrawals = await tokenBridge.getPendingWithdrawals({ recipient });

const transactions = (await Promise.all(pendingWithdrawals.pending.map(async (withdrawal, index) => {
const tokenMapping = await tokenBridge.getTokenMapping({

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

nice this is how you get the mapping

jeetparikh
jeetparikh previously approved these changes Aug 18, 2025
@keithbro-imx keithbro-imx enabled auto-merge August 19, 2025 00:18
@keithbro-imx keithbro-imx added this pull request to the merge queue Aug 19, 2025
Merged via the queue into main with commit cce02d6 Aug 19, 2025
8 checks passed
@keithbro-imx keithbro-imx deleted the pending-withdrawals-poc branch August 19, 2025 01:08
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Development

Successfully merging this pull request may close these issues.

3 participants