StockMarket exchange property doesn't use the Exchanges type

[Branch]

Description

When using the StockMarket component I was expecting to be able to use the Exchanges type for the exchange property within this component. Instead I found that the StockMarket component is using a a union type where it has defined its own exchanges. These exchanges are different from the ones used in the Exchanges type.

Exchanges type:

react-ts-tradingview-widgets/src/index.ts

Line 384 in 0995937

export type Exchanges =

StockMarket union:
https://github.com/JorrinKievit/react-ts-tradingview-widgets/blob/09959377b501b44d3cfef4c113a14c33e00c5c1d/src/components/StockMarket.tsx#L8C3-L8C3

@JorrinKievit Is this done on purpose? I also noticed that none of them match the exchanges found on for example https://www.tradingview.com/widget/market-movers/ - this is causing issues when using the StockMarket component.

[JorrinKievit]

Most of them match right? There are just quite a lot missing from the list. 😄

Why it was done? Its been a long time so I'm not too sure about my decision-making 😅 , but I guess because there were different exchange choices between StockHeatMap (only place the Exchanges from index.ts is used) and StockMarket widget when it was implemented.

[Branch]

I understand! I asked because I wasn't sure how to best solve it, if we should consolidate all exchanges into one type that all components that are using an exchange prop should use or not.

[JorrinKievit]

If both components use the same exchanges, then that will be good! If not, maybe its best to keep exchange types in their component instead of globally

[Branch]

Agree, I'll see if I can look into it later this week.

[JorrinKievit]

Thank you for your effort 😄