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

Modify ConfigurationDTO to generalize Json files export and import #223

Open
lufe089 opened this issue Aug 31, 2017 · 0 comments
Open

Modify ConfigurationDTO to generalize Json files export and import #223

lufe089 opened this issue Aug 31, 2017 · 0 comments

Comments

@lufe089
Copy link
Contributor

lufe089 commented Aug 31, 2017

I'm submitting a...


[ ] Regression (a behavior that used to work and stopped working in a new release)
[ ] Bug report  
[ x] Feature request
[ ] Documentation issue or request
[ ] Support request => Please do not submit support request here

Current behavior

The monitor module saves and reads json files which are saved by using the class ConfigurationIO.

Expected behavior

Inside the ConfigurationIO class there are methods to handle json files (load, save, read) that could be generalized to be used for other modules (different from the Configuration tasks).
Improve how these methods were developped to obtain a generic design.
The idea is that the IO module does not have any relation with any module that contains business logic. Only the dependency with the common module is allowed.

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

No branches or pull requests

1 participant