Gateway connect LP connectors - Autocomplete for `chain` displays connected LP connectors

hummingbot / gateway

Middleware that standardizes DEX API endpoints on different blockchain networks

Apache License 2.0

73 stars 122 forks source link

Gateway connect LP connectors - Autocomplete for `chain` displays connected LP connectors #284

Closed rapcmia closed 3 weeks ago

rapcmia commented 6 months ago

Describe the bug After you successfully connected wallets for a LP connector and decided to add again for a different chain, the autocomplete displays the other LP connectors resulting as option for users to select.

This results to errors chain not supported and invalid network when setting up LP connectors

Steps To Reproduce

Add uniswapLP both ethereum and arbitrum chain
Add pancakeswapLP and when selecting chain to connect, press tab or wait for the autocomplete to show

Release version dev-1.25.0

Attachments n/a

nikspz commented 6 months ago

Confirmed

connected LPconnectors showed as chains at uniswapLP or PancakeswapLP

nikspz commented 6 months ago

Changed title since showed only for LP connectors

nikspz commented 1 month ago

Reproduced in https://github.com/hummingbot/gateway/pull/340

vic-en commented 1 month ago

@nikspz I think this is an existing bug. It couldn't have been introduced in my 1 line update pr on the client.

nikspz commented 3 weeks ago

Reproduced on dev-2.0.1 version

nikspz commented 3 weeks ago

Bounty:

Sponsor: HB Foundation (Connector pots)
Bounty amount: 10K HBOT
Developer portion: 10K HBOT

isreallee82 commented 3 weeks ago

i can look into this @nikspz

cybersecuritylearning commented 3 weeks ago

I can take a look into this @nikspz