Skip to content

Conversation

@mixplore
Copy link
Contributor

@mixplore mixplore commented Nov 27, 2025

⚠️ to be merged after #460

Description

We weren't properly supporting NXM as payment asset in the SDK (using coverAsset and premiumInAsset values from the quote res). This PR adds a fix for it, supporting NXM as payment asset for buy cover.

Testing

Checklist

  • Performed a self-review of my own code
  • Made corresponding changes to the documentation

@mixplore mixplore requested a review from MilGard91 November 27, 2025 11:18
@mixplore mixplore self-assigned this Nov 27, 2025
@mixplore mixplore force-pushed the fix/quote-payment-nxm branch from f8663f3 to abd4af7 Compare November 27, 2025 11:33
@cloudflare-workers-and-pages
Copy link

cloudflare-workers-and-pages bot commented Nov 27, 2025

Deploying sdk with  Cloudflare Pages  Cloudflare Pages

Latest commit: 4da3a1e
Status: ✅  Deploy successful!
Preview URL: https://5059d85e.sdk-9yp.pages.dev
Branch Preview URL: https://fix-quote-payment-nxm.sdk-9yp.pages.dev

View logs

@mixplore mixplore changed the base branch from dev to feat/26112025-new-listings November 27, 2025 11:33
MilGard91
MilGard91 previously approved these changes Nov 28, 2025
Base automatically changed from feat/26112025-new-listings to dev November 28, 2025 09:05
@mixplore mixplore dismissed MilGard91’s stale review November 28, 2025 09:05

The base branch was changed.

@mixplore mixplore force-pushed the fix/quote-payment-nxm branch from abd4af7 to 4da3a1e Compare November 28, 2025 09:06
@mixplore mixplore merged commit 4e97448 into dev Nov 28, 2025
2 checks passed
@mixplore mixplore deleted the fix/quote-payment-nxm branch November 28, 2025 09:17
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.

3 participants