search

chop-dbhi / omop_harvest

OMOP CDMV4 data model with OSIM2 data in Harvest
https://eig.research.chop.edu/omop/
Other
5 stars 3 forks source link

Issue 25 #27

Closed lcdavidson closed 10 years ago

lcdavidson commented 10 years ago

Updates to the comments and PR are forthcoming.

gracebrownecodes commented 10 years ago

From the file changes in this PR, I'm not entirely clear how you fixed #25. Can you describe how you expect the internal and external deployment process to differ and how the code changes here will result in those different processes ending up with a different set of installed apps?

naegelyd commented 10 years ago

These 3 commits seem to all have the same commit message. Please squash them into a single commit before this PR is merged.

lcdavidson commented 10 years ago

Affirmed.