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

Rework the configuration file #22

Open
NathanNr opened this issue Jan 1, 2018 · 4 comments
Open

Rework the configuration file #22

NathanNr opened this issue Jan 1, 2018 · 4 comments
Assignees
Labels
improvement Improvement to a feature, directly affecting either end- or API-users
Milestone

Comments

@NathanNr
Copy link
Collaborator

NathanNr commented Jan 1, 2018

Features wich are set to true should be always allowed, no differentiation because of the LabyMod default settings; auto-update from the old configuration file.
Maybe split the configuration to two files, one for the LabyMod features and one for the other stuff.

@NathanNr NathanNr added the enhancement Enhancement to a feature not visible to end- or API-users (Such as cleanups of redundant tasks) label Jan 1, 2018
@NathanNr NathanNr added this to the version 2.0 milestone Jan 1, 2018
@heisluft heisluft added improvement Improvement to a feature, directly affecting either end- or API-users and removed enhancement Enhancement to a feature not visible to end- or API-users (Such as cleanups of redundant tasks) labels Jan 7, 2018
@heisluft heisluft mentioned this issue Apr 5, 2018
@NathanNr
Copy link
Collaborator Author

NathanNr commented Apr 7, 2018

Todo: create a new configuration reader and use it, update the configuration updater

@NathanNr
Copy link
Collaborator Author

The configuration reader API has been reworked.
Todo: create the configuration file updater

NathanNr added a commit that referenced this issue Jan 28, 2019
@NathanNr
Copy link
Collaborator Author

Todo: test and use the new ConfigUpdater

NathanNr added a commit that referenced this issue Jan 29, 2019
NathanNr added a commit that referenced this issue Jan 29, 2019
@heisluft
Copy link
Member

@NathanNr Have you tested the config updater yet?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
improvement Improvement to a feature, directly affecting either end- or API-users
Projects
None yet
Development

No branches or pull requests

2 participants