ConradSelig / cliStocksTracker

A command line utility for tracking a stock market portfolio. Primarily featuring high resolution braille graphs.
Apache License 2.0
55 stars 14 forks source link

Load Stocks Independently from Market Sync #35

Closed patrick-hovsepian closed 3 years ago

patrick-hovsepian commented 3 years ago
ConradSelig commented 3 years ago

I like where this is headed - and with the context that an in-line command line will be present helps explain the removal of that config.ini. However, I'm thinking it might be smart to wait to pull this one until that pull request is ready - otherwise the only way to change the configuration is by command line argument, something that can be quite painful when changing a lot of the setting in config.ini.

Another question - will it be possible for users to set their own defaults? I personally don't want to have the change the width / height of the graph, the graphing mode, and the timezone everytime I run the program.

patrick-hovsepian commented 3 years ago

to wait to pull this one until that pull request is ready

sure but its going to be a giant PR and wont be ready for a bit. i'll figure something out for defaults too by then