Skip to content

Attempt fetching Axelar status when Squidrouter API fails.#1089

Open
gianfra-t wants to merge 4 commits intostagingfrom
use-axelar-status-as-fallback-for-squidrouter-pay
Open

Attempt fetching Axelar status when Squidrouter API fails.#1089
gianfra-t wants to merge 4 commits intostagingfrom
use-axelar-status-as-fallback-for-squidrouter-pay

Conversation

@gianfra-t
Copy link
Contributor

@gianfra-t gianfra-t commented Mar 4, 2026

  • Adds the settlementSubsidy -> destinationTransfer sub-flow to Monerium onramps.
  • Adds a fallback to squidRouterPay phase in case Squidrouter's API is not able to find the bridge process, even if it exists.

@netlify
Copy link

netlify bot commented Mar 4, 2026

Deploy Preview for vortexfi ready!

Name Link
🔨 Latest commit f3c05ea
🔍 Latest deploy log https://app.netlify.com/projects/vortexfi/deploys/69a9fe6e4298ff000836aa17
😎 Deploy Preview https://deploy-preview-1089--vortexfi.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify project configuration.

@netlify
Copy link

netlify bot commented Mar 4, 2026

Deploy Preview for vortex-sandbox ready!

Name Link
🔨 Latest commit f3c05ea
🔍 Latest deploy log https://app.netlify.com/projects/vortex-sandbox/deploys/69a9fe6ed47fa000089998b3
😎 Deploy Preview https://deploy-preview-1089--vortex-sandbox.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify project configuration.

}

// Map Axelar status to SquidRouter format, assuming GMP transaction.
const mappedStatus =
Copy link
Contributor Author

Choose a reason for hiding this comment

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

Setting this status directly as axelarScanStatus for any other case than success may not be ideal, but is the best we can do in this scenario.

? "success"
: axelarScanStatus.status;

return {
Copy link
Member

Choose a reason for hiding this comment

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

I know it's not super relevant as we wouldn't use the other things anyway, but let's put id: '', squidTransactionStatus: '' and routeStatus: [] instead of leaving them undefined.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants