search

jnanaswaroop / migration_final

Enlightening healthcare since 2012
0 stars 0 forks source link

ph_22927 - [merged] #50

Closed jnanaswaroop closed 2 weeks ago

jnanaswaroop commented 9 months ago

In GitLab by @mjaskiewicz1 on Feb 7, 2024, 03:02

_Merges ph22927 -> master

This ticket originally is about adding a foreign key from organization to acct_facility. I expanded it to include prod_productsku and po_poterm (both to contract table). I didn't add foreign keys originally because the tables were maintained via replication so I didn't want to touch them. We had an error with a bogus facility though so I changed my mind. I had to update the tests as a consequence.

jnanaswaroop commented 9 months ago

In GitLab by @mjaskiewicz1 on Feb 7, 2024, 03:11

added 1 commit

Compare with previous version

jnanaswaroop commented 9 months ago

In GitLab by @mjaskiewicz1 on Feb 7, 2024, 03:22

added 1 commit

Compare with previous version

jnanaswaroop commented 9 months ago

In GitLab by @mjaskiewicz1 on Feb 7, 2024, 22:07

added 1 commit

Compare with previous version

jnanaswaroop commented 9 months ago

In GitLab by @mjaskiewicz1 on Feb 7, 2024, 23:40

added 1 commit

Compare with previous version

jnanaswaroop commented 9 months ago

In GitLab by @mjaskiewicz1 on Feb 7, 2024, 23:55

requested review from @onmay16 and @nkessler

jnanaswaroop commented 9 months ago

In GitLab by @nkessler on Feb 8, 2024, 01:05

mentioned in commit 35ce39a70941b34cb3be009e6eac9ab9ed603568