Refactor to move some logic from the CosmosGateway route to the cosmos context #941
Conversation
This file contains 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
|
New and removed dependencies detected. Learn more about Socket for GitHub ↗︎
🚮 Removed packages: @injectivelabs/sdk-ts@1.0.211, @terra-money/terra.js@3.1.10 |
anondev2323
approved these changes
Sep 11, 2023
M-Picco
force-pushed
the
refactor/gateway-cosmos-context
branch
from
ec042d0
to
f87367f
Compare
…e cosmos context. Remove some sdk unneeded dependencies
M-Picco
force-pushed
the
refactor/gateway-cosmos-context
branch
from
f87367f
to
c8f6dc6
Compare
anondev2323
pushed a commit
that referenced
this pull request
Sep 11, 2023
…e cosmos context. Remove some sdk unneeded dependencies (#941)
anondev2323
pushed a commit
that referenced
this pull request
Sep 12, 2023
…e cosmos context. Remove some sdk unneeded dependencies (#941)
derpy-duck
pushed a commit
that referenced
this pull request
Sep 12, 2023
* fix: standardize route enum naming and create type * fix: type errors * fix: type error and install * fix: circular import * route logic and organize network configs * add: getMinSendAmount on routes * fix: imports and type errors * fix: forgot to commit type fixes * fix: deal with possibly undefined values * fix: amount input * refactor: gas estimation logic * refactor: gas estimation logic * fix: RouteOperator export/import * fix: available route logic * fix: await promise, hide native gas if none * fix: validate route and get available routes * add: supportedChain logic for routes * fix: address TODOs for claim gas estimations * remove comment * fix: circular import * debug: gas estimate shows as 0 * rename to chain * fix: format gas units correctly * refactor: delete getTokenBalance and getNativeBalance from routes * fix: bug in amount input * fix: pass in env rpcs * enhance: supportedSourceTokens logic * fix: rpc fields getting overridden in Object.assign * fix: random bug fixes * add: balances cache * fix: error in addToWallet component * fix: remove check for bridge/relay source tokens and add validation * Move getForeignAsset gateway logic from the CosmosGateway route to the cosmos context. Remove some sdk unneeded dependencies (#941) * fix: rebase errors * clean: imports * fix: duplicate relay options listed instead of bridge/relay * fix: check AUTOMATIC_TRANSFER instead of payload ID * fix: add getTxGasUsed to Solana context * lint * Add missing types to ErrorBoundary --------- Co-authored-by: Martin Picco <martin@xlabs.xyz>
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.
Start with
getForeignAsset