-
Notifications
You must be signed in to change notification settings - Fork 0
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
Adding delivery + new modification bp + updating database structure #34
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Mainly cosmetic request about commeted lines. Can be merge after update.
I finished going over the pull request, and here is a summary of what I understand that this will do. I am not an expert on some of these libraries, so I might be wrong. If the description here is correct, then I will approve the PR so we can move forward while PO is away:
|
That's correct @pphector. |
Perfect, then on my end, I will approve. Although I think it would still help if someone else can take a quick Look before we deploy. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This pull request makes major modifications to the database schema using the alembic library. The changes are versioned, this revision will have a specific ID to allow for potential recovery if required.
The biggest changes involve the change of the patient
table to specimen
. This involves also changing several other column IDs in other tables. Other changes include the addition of a reference
table and ext_id
and ext_src
columns.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This is good to go, I'll let you decide id you want to implement the enumerate on this pull request or sometime downstream
No description provided.