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.

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

Dexalot - EUROC and sAVAX balances are not recognized #232

Closed
rapcmia opened this issue Nov 14, 2023 · 2 comments
Closed

Dexalot - EUROC and sAVAX balances are not recognized #232

rapcmia opened this issue Nov 14, 2023 · 2 comments
Assignees
Labels
bug Something isn't working dexalot connector P3

Comments

@rapcmia
Copy link
Contributor

rapcmia commented Nov 14, 2023

Describe the bug
EUROC and sAVAX are not being recognized on Dexalot exchange when used. Tried to add these to the gateway connector-token command but still no go.

On the screenshot below, ran balance command and observed only few of the added tokens are showing available yet on exchange it has balance. The tokens sAVAX and EUROC are also updated on avalanche_tokens.json
image

When starting the strategy using DEX connector, it is expected to display the balances for the market pair but no avail

  • This causes the exchange to only submit order on buy which at the time it was running, it has enough balance to open a sell order
    image
    image

Steps To Reproduce

  • Setup client and gateway
  • Setup dexalot wallet then check balances by:
    • Using gateway balance command
    • Using gateway connector-tokens
    • Setup PMM then start, observe the prompt for balances

Release version
dev-1.22.0

logs_dexalot-eurocusdc.log

@rapcmia rapcmia added bug Something isn't working dexalot connector P3 labels Nov 14, 2023
@rapcmia rapcmia moved this to Not started in Issues - Connector Nov 14, 2023
@fengtality
Copy link
Contributor

My guess is this check in https://github.com/hummingbot/gateway/blob/main/src/connectors/dexalot/dexalot.ts#L136
is the issue:

    const balances = await Promise.all(
      req.tokenSymbols.map((symbol) => {
        return this._portfolioContract.getBalance(
          req.address,
          fromUtf8(symbol)
        );
      })
    );

@rapcmia rapcmia moved this from Not started to In review in Issues - Connector Nov 20, 2023
@vic-en vic-en mentioned this issue Jan 16, 2024
2 tasks
@rapcmia
Copy link
Contributor Author

rapcmia commented Jan 17, 2024

Fix applied to development branch.

@rapcmia rapcmia closed this as completed Jan 17, 2024
@github-project-automation github-project-automation bot moved this from In review to Closed in Issues - Connector Jan 17, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working dexalot connector P3
Projects
Status: Closed
Development

No branches or pull requests

2 participants