Skip to content
This repository has been archived by the owner on Feb 15, 2022. It is now read-only.

Minor Changes + Poloniex Currencies #19

Closed
wants to merge 61 commits into from

Conversation

ghostnegotiator
Copy link

@ghostnegotiator ghostnegotiator commented Aug 15, 2016

Changes:

  • Added config.js to gitignore
  • Renamed config.js to sample-config.js
  • Added a few Crypto currencies for Poloniex...

Soon to be done:
Adding the rest of currencies at Poloniex plugin...

This is done with allot of <3

@carlos8f
Copy link
Contributor

Hey @RDash21 , I committed your poloniex json, but for the config stuff, right now I'm leaning to have config.js be git-controlled so I can change config vars as development evolves. Config vars become obsolete or need new defaults often. I used to have a sample-config.js but as I updated it I had to manually update the real config.js frequently and it's a pain to keep in sync.

@ghostnegotiator
Copy link
Author

That's understandable, how about making it possible to select your config trough a '--config' '-c' flag ? :)

@ghostnegotiator
Copy link
Author

More products will be added by tomorrow*

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants