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

Feat/add db tables #3

Open
wants to merge 32 commits into
base: develop
Choose a base branch
from
Open

Feat/add db tables #3

wants to merge 32 commits into from

Conversation

mikevespi
Copy link
Contributor

Pull #2 has to be merged before this, then this will need a rebase.
Pull to begin adding tables to the DB.

@tmastrom tmastrom self-assigned this Aug 31, 2022
@MichaelVeniez
Copy link

MichaelVeniez commented Nov 9, 2022

@tmastrom what is the status of this card?

@mikevespi
Copy link
Contributor Author

@MichaelVeniez it will be an ongoing process. We haven't established a solid ERD yet as we're still waiting for information on a certain data set... It has some things that will stay, and some things that will go. Dylan and I decided to keep the PR alive so that we can just strip out tables we don't need rather than start a new PR and rewrite everything that's going to stay. The card itself likely won't be "finishable" until we have more information. The schema won't hit its final for a while, but once we have the basic tables we know we need in, we can call it done.

@mikevespi
Copy link
Contributor Author

The above comment holds true, but I am putting it to the development team. If you'd prefer to continue the existing branch and clean out what isn't needed, add what is, you are free to. If you'd prefer to create new branches and smaller tickets, that is also viable. Make sure to grab the necessary functions and policies (something the developers should sync about) out of the existing PR.

@Samiabutton Samiabutton mentioned this pull request Jan 20, 2023
9 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants