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.

Sign up for GitHub

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

Jump to bottom

Config file handling #11

Open
wants to merge 3 commits into
base: master
Choose a base branch
from
Open

Config file handling #11

wants to merge 3 commits into from

Commits on Aug 9, 2016

  1. Async and sync config handling 

    Config handling should be async only on load (or on refresh). For this
I've made the following changes/additions:
1) The graph is not drawn on load - only when done loading the config
file (or on failure).
2) The config is requested in the Factory's constructor - if request
fails, asks for defaultConfig.
3) Added local method defaultConfig.
4) Added prototype method: getConfig
5) initLayout uses existing config (should explicitely use
"refreshConfig" in order to change the
6) Added prototype method: "refreshConfig" which fetches config from the
config file.
    @YonatanKra
    YonatanKra committed Aug 9, 2016
    Configuration menu
    Copy the full SHA
    df0fe99 View commit details
    Browse the repository at this point in the history

  2. Merge remote-tracking branch 'refs/remotes/Webiks/master' into yonata… 

    …n-kra-patches
    @YonatanKra
    YonatanKra committed Aug 9, 2016
    Configuration menu
    Copy the full SHA
    1876412 View commit details
    Browse the repository at this point in the history

  3. [Fix]: ajax response handler now accesses the correct data 

    There was an error when accessing the Ajax response. Was trying to get
config directly from "response" but should have been "response.data"
    @YonatanKra
    YonatanKra committed Aug 9, 2016
    Configuration menu
    Copy the full SHA
    386c162 View commit details
    Browse the repository at this point in the history