fix / Support latest PolkaDEX API

[dizpers]

Before submitting this PR, please make sure:

• Your code builds clean without any errors or warnings
• You are using approved title ("feat/", "fix/", "docs/", "refactor/")

A description of the changes proposed in the pull request:

PolkaDEX team had updated their APIs (with breaking changes), so the connector in public version of Hummingbot became broken. This PR made all the fixes needed for this connector to work properly and also include some improvements (like result pagination).

The test suite still need to be updated, so I'm disabling it for now.

Tests performed by the developer:

1. Manually tested in local environment (following HF checklist)
2. Bots are successfully running in production environment.

Tips for QA testing:

N/A

[nikspz]

@dizpers thank you for submitting PR, fyi, it should be aimed to development, I changed it for now https://hummingbot.org/developers/contributions/#checklist

[rapcmia]

Hi @dizpers good day
Can you take a look on the branch conflicts? thanks

[dizpers]

@nikspz @rapcmia Thanks for initial checks, working on conflicts resolution now.

[dizpers]

New API docs: https://app.theneo.io/polkadex/orderbook-api-docs/api-reference

[dizpers]

@rapcmia @nikspz conflicts are resolved now

[dizpers]

Looks like it requires tests to be fixed as well

[rapcmia]

LGTM

• Ran tests on source build local macos
• Create new account and use for setting up the connector, all ok
• Balance displayed ok
• Using v1 strategy (pmm)
  • Orders are created, fetched and cancelled successfully
  • Trade orders are recorded matched between history, CSV and exchange trade history
  • Run for couple of hours with low and high refresh time ok
• Build docker image ok

[notion-workspace]

fix / Support latest PolkaDEX API #7016

[notion-workspace]

fix / Support latest PolkaDEX API #7016

[dizpers]

@rapcmia All PolkaDEX tests has been fixed. So I'm enabling them back in Makefile and in coverage configuration.

[Gauthamastro]

Hey guys, is there anything blocking this to be merged? Can we help somehow to merge this sooner?

[nikspz]

hi @Gauthamastro Yes, we need to add test coverage here

@dizpers
https://github.com/hummingbot/hummingbot/actions/runs/9007313287/job/24764466360?pr=7016

[dizpers]

@Gauthamastro @nikspz yes, Enflux team is working on that now.

[dizpers]

@rapcmia @nikspz coverage should be 80%+ now, could your run the workflow?

[rapcmia]

Hi @dizpers thanks for the update, the github checks are failing on some tests below:

[notion-workspace]

fix / Support latest PolkaDEX API #7016

[dizpers]

@rapcmia @nikspz could you run the pipeline one more time?

[cardosofede]

LGTM!

[cardosofede]

only supports 2 orders every 2 secs?

[rapcmia]

Merged to development and part of release version 1.28.0