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/remove hotbit #6359

Merged
merged 4 commits into from Jun 5, 2023
Merged

Conversation

cardosofede
Copy link
Contributor

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:
Remove Hotbit from the codebase based on the last poll

Tests performed by the developer:

Tips for QA testing:

@cardosofede cardosofede changed the base branch from master to development June 2, 2023 11:31
Copy link
Contributor

@david-hummingbot david-hummingbot left a comment

Choose a reason for hiding this comment

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

Source Build

  • source build successful
  • launch client + password login successful
  • run connect command and verified hotbit is removed from list and autocomplete
  • tried to start strategy with hotbit and checked if connector is showing in the suggested list

Docker Build

  • source build successful
  • launch client + password login successful
  • run connect command and verified hotbit is removed from list and autocomplete
  • tried to start strategy with hotbit and checked if connector is showing in the suggested list

2023-06-05_083307
2023-06-05_083241

Copy link
Contributor

@nikspz nikspz left a comment

Choose a reason for hiding this comment

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

Test performed:

  • Cloned and installed Source build successfully
  • Verified hotbit is removed from connect list and strategy creation autocomplete
  • Manually built docker image successfully
  • Created docker instance and review hotbit removed
  • Review hotbit files was removed

@nikspz nikspz merged commit 9963761 into hummingbot:development Jun 5, 2023
3 checks passed
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.

None yet

3 participants