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

update proxy and candles urls #83

Closed
wants to merge 1 commit into from
Closed

update proxy and candles urls #83

wants to merge 1 commit into from

Conversation

smk762
Copy link
Contributor

@smk762 smk762 commented Nov 13, 2023

Just a little housekeeping.

komodo.earth:3334 -> etherscan-proxy.komodo.earth
komodo.earth:3333 -> candles.komodo.earth

Old urls still work, for now. This repo is the last one still using the old ones. No rush on review / merge, but should be included in the next release.

To test:

  • confirm candle charts still display
  • confirm evm trasaction histories are updating correctly.

@smk762 smk762 requested review from CharlVS and kivqa November 13, 2023 11:44
@smk762 smk762 unassigned CharlVS and kivqa Nov 13, 2023
@smk762 smk762 changed the base branch from master to dev November 13, 2023 11:45
@CharlVS
Copy link
Member

CharlVS commented Nov 20, 2023

@kivqa Everything working as expected?

@CharlVS CharlVS added the QA Ready for QA testing label Nov 20, 2023
@kivqa
Copy link

kivqa commented Nov 20, 2023

Checking of this PR blocked by issue with API. https://candles.komodo.earth/api/v1/ohlc/eth-btc and https://komodo.earth:3333/api/v1/ohlc/eth-btc returns null

@CharlVS
Copy link
Member

CharlVS commented Dec 17, 2023

We are still awaiting to find an alternate candlestick provider. The new provider is no longer available.

@CharlVS CharlVS marked this pull request as draft December 17, 2023 08:45
@smk762
Copy link
Contributor Author

smk762 commented Jan 17, 2024

https://binance-docs.github.io/apidocs/spot/en/#kline-candlestick-streams is an option with wide support for coins, though due to their API limit policies, this should be implemented client side.

@CharlVS
Copy link
Member

CharlVS commented Mar 15, 2024

Due to an issue with the new API provider, this was abandoned in favour of implementing Binance's API, which includes some minor improvements/fixes by @takenagain.

#117

@CharlVS CharlVS closed this Mar 15, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
QA Ready for QA testing
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants