feat: add Edge exchange provider API requirements#3347
Open
TaprootFreak wants to merge 1 commit intodevelopfrom
Open
feat: add Edge exchange provider API requirements#3347TaprootFreak wants to merge 1 commit intodevelopfrom
TaprootFreak wants to merge 1 commit intodevelopfrom
Conversation
- Add evmChainId to AssetInDto for EVM chain identification - Add EvmUtil.getBlockchain() reverse lookup (chainId → Blockchain) - Extend resolveAsset() to support evmChainId + chainId combination - Add RegionRestricted and AssetUnsupported to QuoteError enum - Add structured errors array to Buy/Sell/Swap quote DTOs - Add QuoteErrorUtil to map internal errors to Edge format - Add GET /transaction/status/:orderId with simplified status mapping - Update EDGE_REQUIREMENTS.md tracking
Member
Open points (WIP)1. Chain and Token Identification ✅
2. Order Identification and Status Page ⏳
3. Error Handling
4. Quoting Requirements ✅ 5. Transaction Status API ✅
6. Reporting API
7. Account Activation ⛔
8. Affiliate Revenue Withdrawal
9. User Authentication
10. Regional and Fiat Currency Support
11. KYC Information
12. Bank Information
13. Verification
14. Widgets
15. Off-Ramp Flow
|
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary
evmChainIdfield toAssetInDtowithEvmUtil.getBlockchain()reverse lookup, enabling asset resolution via numeric EVM chain ID + contract addresserrorsarray to all quote DTOs (Buy/Sell/Swap) with Edge-compatible error codes (RegionRestricted,OverLimitError,UnderLimitError) includingsourceAmountLimit/destinationAmountLimitGET /transaction/status/:orderIdendpoint mapping 14 internal states to 6 Edge-format statuses (pending,processing,infoNeeded,expired,refunded,completed)Changed files
asset.dto.tsevmChainIdtoAssetInDto, adjust validationevm.util.tsgetBlockchain()reverse lookuppayment-info.service.tsresolveAsset()with evmChainId logicquote-error.enum.tsREGION_RESTRICTED,ASSET_UNSUPPORTEDstructured-error.dto.tsquote-error.util.tsbuy-quote.dto.ts/sell-quote.dto.ts/swap-quote.dto.tserrorsfieldbuy.controller.ts/sell.controller.ts/swap.controller.tserrorsin quote responsestransaction-status.dto.tstransaction.controller.tsGET /transaction/status/:orderIdEDGE_REQUIREMENTS.mdTest plan
evmChainIdin Swagger UI for AssetInDtoevmChainId+chainId(withoutblockchain)errorsarray appears in Buy/Sell/Swap quote responsesGET /transaction/status/:orderIdwith valid uid/orderUid