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

(feat) TELOS integration #338

Merged
merged 1 commit into from
Jul 29, 2024

Conversation

evseevnn
Copy link
Contributor

@evseevnn evseevnn commented Jul 5, 2024

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:

Tests performed by the developer:

  • Tested by running locally and making requests using curl.
  • Tested with Hummingbot
  • Added tests for TELOS routes in OpenOcean.

Tips for QA testing:

@evseevnn evseevnn changed the title TELOS integration feat/ TELOS integration Jul 5, 2024
@evseevnn evseevnn changed the title feat/ TELOS integration (feat) TELOS integration Jul 9, 2024
@rapcmia rapcmia requested a review from fengtality July 23, 2024 05:01
@nikspz nikspz self-requested a review July 23, 2024 19:32
@nikspz
Copy link
Contributor

nikspz commented Jul 23, 2024

Test performed:

  • Created telos wallet, added deposit
  • connected openocean telos evm wallet successfully
  • review gateway balance is correct
  • added WTLOS,USDC to connector tokens
  • review tokens added and show correct balance
  • approve token (USDC) through the client - failed, have to approved manually
  • amm_arb created successfully using openocean telos evm
  • started and review status showed correct prices
  • after some time got error price quiery request failed, same on curl

logs_tlos.log

image
image

image

@nikspz
Copy link
Contributor

nikspz commented Jul 24, 2024

Copy link
Contributor

@fengtality fengtality left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@nikspz nikspz merged commit 6c55d4f into hummingbot:development Jul 29, 2024
2 of 3 checks passed
@nikspz nikspz mentioned this pull request Aug 12, 2024
2 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants