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

Integrated Coingi and added a new feature "exchangeMaxHistoryAge" #2018

Merged
merged 5 commits into from Mar 12, 2018

Conversation

Projects
None yet
2 participants
@jmcklondonuk
Copy link
Contributor

jmcklondonuk commented Mar 9, 2018

  • What kind of change does this PR introduce? (Bug fix, feature, docs update, ...)

feature

  • What is the current behavior? (You can also link to an open issue here)

Supported exchanges don't cointain coingi. Data import is only supported where exchanges return their full data.

  • What is the new behavior (if this is a feature change)?
    Coingi is among supported exchanges. Data import is supported also when exchanges return only the last X days (for example the last 30 days)

  • Other information:
    https://forum.gekko.wizb.it/thread-1447.html

jmcklondonuk added some commits Mar 8, 2018

@askmike

This comment has been minimized.

Copy link
Owner

askmike commented Mar 12, 2018

Great! Thanks a lot!

@askmike askmike merged commit da7519c into askmike:develop Mar 12, 2018

2 checks passed

continuous-integration/appveyor/pr AppVeyor build succeeded
Details
continuous-integration/travis-ci/pr The Travis CI build passed
Details

This was referenced Mar 21, 2018

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
You can’t perform that action at this time.