Skip to content

Conversation

@rharutyunyan
Copy link
Contributor

@rharutyunyan rharutyunyan commented Sep 15, 2025

Change Summary

What does this PR change?
Add native asset support for cross chain sdk

Related Issue/Ticket:

Testing & Verification

How was this tested?

  • Unit tests
  • Integration tests
  • Manual testing (describe steps)
  • Verified on staging

Risk Assessment

Risk Level:

  • Low - Minor changes, no operational impact
  • Medium - Moderate changes, limited impact, standard rollback available
  • High - Significant changes, potential operational impact, complex rollback

Risks & Impact

@rharutyunyan rharutyunyan marked this pull request as ready for review September 17, 2025 12:05
enablePermit2: params.isPermit2
}

return this._createOrderEvmOrder(
Copy link
Member

Choose a reason for hiding this comment

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

_createOrderEvmOrder -> _createEvmOrder

pagination?: PaginationRequest
public async getCancellableOrders(
chainType: ChainType,
pagination: PaginationRequest
Copy link
Member

Choose a reason for hiding this comment

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

make pagination optional

@krboktv krboktv merged commit 9415d61 into master Sep 29, 2025
1 check failed
@krboktv krboktv deleted the feat/native-asset-support branch September 29, 2025 11:56
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