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

Database package location refactor #1945

Closed
Samyoul opened this issue Apr 20, 2020 · 2 comments
Closed

Database package location refactor #1945

Samyoul opened this issue Apr 20, 2020 · 2 comments

Comments

@Samyoul
Copy link
Member

Samyoul commented Apr 20, 2020

Problem

This repository has a number of database related packages that are spread out across the code base and I'm looking to consult with the team about consolidating them.

The other major problem with this implementation is that the business logic is too tightly coupled to the DB type and syntax with no layer of abstraction in between. This would make changing to another form of data persistence an involved task.

Packages I've identified are:

Implementation

  1. Discuss the roles of each of the identified packages, identify any additional packages that are database related.
  2. Come to a consensus about the consolidation approach for the identified packages.
  3. Move packages into the decided structure
  4. Refactor all dependant packages

Acceptance Criteria

Database related packages are restructured as per the consensus arrived to here in this issue.

Notes

This isn't a witch hunt, or in any way a criticism of implementation choices made in the past, I'm certain every contributor that placed these packages where they are currently had legitimate reasons for doing so. The purpose of this issue is to get the code base into a state where it is more easily understood by a new developer (like me 😄 ), and consolidate related packages where it makes sense.

Future Steps

Any additional database based functionality should adhere to the conventions established here in this issue.

@Samyoul
Copy link
Member Author

Samyoul commented Apr 20, 2020

@cammellos FYI, what we were discussing earlier today.

@status-github-bot
Copy link

This issue has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs. Thank you for your contributions.

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

2 participants