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

Aggregators (e.g., CoinMarketCap, CionGecko) id for currencies #22171

Open
1 task done
bozorgmehr96 opened this issue Apr 16, 2024 · 2 comments
Open
1 task done

Aggregators (e.g., CoinMarketCap, CionGecko) id for currencies #22171

bozorgmehr96 opened this issue Apr 16, 2024 · 2 comments
Assignees

Comments

@bozorgmehr96
Copy link

bozorgmehr96 commented Apr 16, 2024

Preliminary Checks

  • I have already searched for existing issues and confirmed that this issue is not a duplicate

Is your feature request related to a problem? Please describe

No response

Describe the solution you'd like

As good as the CCXT unified id is, it's not enough for all scenarios.
Is it possible to add cmcId and coinGeckoId to currencies?
Instead of adding aggregators as Exchange, add their ids to existing exchanges.
Is it less or more work to do?

Describe alternatives you've considered

No response

@Dan-krm Dan-krm self-assigned this Apr 17, 2024
@Dan-krm
Copy link
Contributor

Dan-krm commented Apr 17, 2024

Hi @bozorgmehr96 I'm not sure what having different aggregator ids in the currency structure would be required for in CCXT.

As good as the CCXT unified id is, it's not enough for all scenarios.

Could you share the scenarios where the CCXT unified id is not enough?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

3 participants
@bozorgmehr96 @Dan-krm and others