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

Extract feel repository from storage adapter #2

Open
jgshort opened this issue Oct 18, 2018 · 0 comments
Open

Extract feel repository from storage adapter #2

jgshort opened this issue Oct 18, 2018 · 0 comments
Labels
code debt Improve maintainability enhancement New feature or request

Comments

@jgshort
Copy link
Owner

jgshort commented Oct 18, 2018

As a developer, I want a clean separation of the underlying storage layer from the operations of the data layer. Right now, feels are tightly integrated with the underlying storage adapter (which happens to be sqlite3).

Extract feel-specific data operations from the storage adapter. Create a new 'feel' repository with a dependency on the storage interface. Move feel operations up to the repository.
@jgshort jgshort added enhancement New feature or request code debt Improve maintainability labels Oct 18, 2018
@jgshort jgshort self-assigned this Oct 18, 2018
@jgshort jgshort removed their assignment Oct 18, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
code debt Improve maintainability enhancement New feature or request
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
1 participant
@jgshort