Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

LiquidityAlert #13

Merged
merged 1 commit into from
Jan 18, 2023
Merged

LiquidityAlert #13

merged 1 commit into from
Jan 18, 2023

Conversation

raviansalman
Copy link
Contributor

  • Added the error handling for showing warnings when there is no liquidity for a swap pair on the DEX
  • Updated the weth configuration from Network.js

Note: In MultiSwap codebase, weth is the default name for the native token of each blockchain against a DEX Router, for example,

  • on Polygon DEX Router weth → wrapped MATIC,
  • on BSC DEX Router weth → wrapped BNB,
  • on Ethereum DEX Router weth → wrapped Ethereum

@taha-abbasi taha-abbasi self-requested a review January 18, 2023 21:53
@taha-abbasi taha-abbasi merged commit 5f24774 into ferrumnet:develop Jan 18, 2023
raviansalman added a commit to raviansalman/MultiSwap that referenced this pull request Sep 4, 2024
@raviansalman
SwapOnSameNetwork & SourceNativeSwap (ferrumnet#13)
9a5d94d 
* QP-Stargate-Fee

* few changes

* Config Script Added

* qp-ms integration

* tokenABI

* "minor change"

* SameNetworkSwap/SrcEthSwap
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@SibghatUllah1997 SibghatUllah1997 SibghatUllah1997 approved these changes

@taha-abbasi taha-abbasi taha-abbasi approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@raviansalman @taha-abbasi @SibghatUllah1997