search

jnanaswaroop / migration_final

Enlightening healthcare since 2012
0 stars 0 forks source link

ph_23436 - [merged] #60

Closed jnanaswaroop closed 2 weeks ago

jnanaswaroop commented 5 months ago

In GitLab by @mjaskiewicz1 on Jun 24, 2024, 19:40

_Merges ph_23436_indexmd5 -> master

Adding an index on the md5_key column. Also, we need to copy the vendor and manufactuer_part_number columns over to prod. They columns are ready, and we just need to move the values from contract to staging.

jnanaswaroop commented 5 months ago

In GitLab by @mjaskiewicz1 on Jun 24, 2024, 19:41

requested review from @onmay16 and @nkessler

jnanaswaroop commented 5 months ago

In GitLab by @mjaskiewicz1 on Jun 24, 2024, 21:37

added 1 commit

Compare with previous version

jnanaswaroop commented 5 months ago

In GitLab by @mjaskiewicz1 on Jun 24, 2024, 21:45

added 1 commit

Compare with previous version

jnanaswaroop commented 5 months ago

In GitLab by @mjaskiewicz1 on Jun 24, 2024, 21:51

added 1 commit

Compare with previous version

jnanaswaroop commented 5 months ago

In GitLab by @mjaskiewicz1 on Jun 24, 2024, 21:55

added 1 commit

Compare with previous version

jnanaswaroop commented 5 months ago

In GitLab by @nkessler on Jun 24, 2024, 22:01

nothings jumping out at me and tests pass. lgtm

jnanaswaroop commented 5 months ago

In GitLab by @nkessler on Jun 24, 2024, 22:01

approved this merge request

jnanaswaroop commented 5 months ago

In GitLab by @mjaskiewicz1 on Jun 24, 2024, 22:17

mentioned in commit eed898d9cb0a05119a6fb10f47c262d5e5f39a17