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

Add a data storage class #14

Closed
Tracked by #3
vkbo opened this issue Dec 14, 2021 · 0 comments · Fixed by #15
Closed
Tracked by #3

Add a data storage class #14

vkbo opened this issue Dec 14, 2021 · 0 comments · Fixed by #15
Assignees
Labels
core feature Planned core functionality
Milestone

Comments

@vkbo
Copy link
Owner

vkbo commented Dec 14, 2021

Epic: #3
Related: #4

The reading and writing of the project's JSON files should be done via a storage class. The path to each file within a project can then be handled internally in such a class. Since everything is JSON, the loading of both files and individual values can be standardised to avoid duplication of code.

The benefit if having a single class that handles all file I/O is that it would be easy to extend such a class to allow storing the project data in a zip archive, or switch between folder and zip storage with a flag.

@vkbo vkbo added the core feature Planned core functionality label Dec 14, 2021
@vkbo vkbo added this to the Projects milestone Dec 14, 2021
@vkbo vkbo self-assigned this Dec 14, 2021
@vkbo vkbo mentioned this issue Dec 18, 2021
5 tasks
@vkbo vkbo closed this as completed in #15 Dec 18, 2021
@vkbo vkbo mentioned this issue Jan 14, 2022
10 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
core feature Planned core functionality
Projects
No open projects
Development

Successfully merging a pull request may close this issue.

1 participant