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.
implements skip replacement for squid /route endpoint
This PR implements the skip analog of squid's
/route
endpoint by utilizing Skip'smsgs_direct
endpoint.In order to do this we:
receivedRoute
method on theSkipProcessor
receivedRouteV2
method on theSkipProcessor
SkipRoutePayloadProcessor
Skip does not provide all the same data that Squid does, so we did our best to keep the data format as similar as possible. We'll have to make some updates on the web side to handle these changes though. Most notably:
Some things to cleanup after this PR:
errors
property is set in a pretty hacky way to coerce the skip payload to mimic squid's. this is intentional to reduce the # of breaking internal API changes. We can migrate that after we confirm core functionality works.slippage
is just hard coded. we made sure that the input param for the skip payload is synced with the processor but we should be updating this value all across the codebase. this will be a separate PR as it's not directly related to this functionalityFuture architectural things:
toAddress
toTokenDenom
toChainId
transferType
etc. in the transfer request.