Giter Site home page Giter Site logo

Comments (5)

taoteh1221 avatar taoteh1221 commented on June 1, 2024

Sorry you're getting a bit lost in the config file. This is partially due to reorganizing it over the last few releases. That work is done now, and it should be easier from here onward to find the settings you need with everything properly categorized now.

I had thought about seperating the config into multiple files, but after thinking it over I decided that over-complicates things for new / inexperienced users on configuration usage.

from open_crypto_tracker.

s1-ranjan avatar s1-ranjan commented on June 1, 2024

As of now I found three unwanted/ user-specific sections:
i: Chart and Price Alert
ii: Standard Rss
iii: Portfolio Asserts
Above mentioned sections should be handled from respective files rather than from the main configuration file, this will definitely shorten the main configuration file, moreover in the above list point's i and iii are more user specific in which people will try to define their currencies for the chart along with the including once coin in the list for market observation.
The main problem comes with the update, with each update there are some modification in the main configuration file, which causes some trouble, to edit the whole configuration file so that to list and modify the chart and Portfolio section, with the split in the configuration file there is a less likelihood of attaining those two section as this two section will remain constant unchanged, I hope I am able to convey my thoughts on this.

from open_crypto_tracker.

taoteh1221 avatar taoteh1221 commented on June 1, 2024

As I said earlier, part of the reason for the confusion was the constant refining of those categories (re-categorizing the content and renaming for a better UX). I honestly would love to split the config file up like you said, but the problem arises when new users have multiple config files instead of one config file. It's just not feasible to do.

I'm sorry about re-categorizing the config file on you alot the past few releases, that work is now done. In fact, I will feature-freeze the config structuring, so the structure stays the same from here forward (other than removing stale markets and adding interesting new ones on ocassion to the demo configuration). There are trade offs for everything thing tweaked inside an app, and unfortunately this is as balanced as it can get without over-complocating things.

from open_crypto_tracker.

s1-ranjan avatar s1-ranjan commented on June 1, 2024

I don't find any issues with this split, we are only required to choose the names and the locations of the files wisely such as
i: Chart and Price Alert can be accommodated in chart.php
ii: Standard Rss can be accommodated in rss.php
iii: Portfolio Asserts can be accommodated in asserts.php
Above files (chart.php, rss.php, asserts.php)can be placed in the php folder under app-lib or some other location, without creating multiple config file, and this solution can be a good way to shorten the config file, and moreover it'll not cause any trouble to the new user as this'll not create multiple config file, rather than a single config file with three different branches and a single configure file.
Confusion can arise only if we use terms like chart-config.php or something like that, so if we go with a neat names as I have suggest this'll not cause any issue to the new user.

from open_crypto_tracker.

taoteh1221 avatar taoteh1221 commented on June 1, 2024

I'm very sorry Ranjan, I don't agree it's a good idea to split the config file up into multiple config files (I already thought it over in depth a long time ago). In future releases, you won't have to edit config.php at all...config will be run from a cached version of your customized config that will never change between updates. At this point, config.php will only be used as a 'default' config for new installs, and 'reset' to default (like resetting your phone). I do promise that I won't reorganize the config categorization again though. That will now remain the same going forward.

from open_crypto_tracker.

Related Issues (20)

Recommend Projects

  • React photo React

    A declarative, efficient, and flexible JavaScript library for building user interfaces.

  • Vue.js photo Vue.js

    🖖 Vue.js is a progressive, incrementally-adoptable JavaScript framework for building UI on the web.

  • Typescript photo Typescript

    TypeScript is a superset of JavaScript that compiles to clean JavaScript output.

  • TensorFlow photo TensorFlow

    An Open Source Machine Learning Framework for Everyone

  • Django photo Django

    The Web framework for perfectionists with deadlines.

  • D3 photo D3

    Bring data to life with SVG, Canvas and HTML. 📊📈🎉

Recommend Topics

  • javascript

    JavaScript (JS) is a lightweight interpreted programming language with first-class functions.

  • web

    Some thing interesting about web. New door for the world.

  • server

    A server is a program made to process requests and deliver data to clients.

  • Machine learning

    Machine learning is a way of modeling and interpreting data that allows a piece of software to respond intelligently.

  • Game

    Some thing interesting about game, make everyone happy.

Recommend Org

  • Facebook photo Facebook

    We are working to build community through open source technology. NB: members must have two-factor auth.

  • Microsoft photo Microsoft

    Open source projects and samples from Microsoft.

  • Google photo Google

    Google ❤️ Open Source for everyone.

  • D3 photo D3

    Data-Driven Documents codes.