openSNP / snpr

The sources of the openSNP website
http://opensnp.org
MIT License
169 stars 46 forks source link

add unique key constraint on Phenotype table #304

Open raivivek opened 8 years ago

raivivek commented 8 years ago

Following from discussion on gitter.

raivivek commented 8 years ago

@gedankenstuecke I am going to assume that since we have a validation step at the controller stage there will be no duplicate entries in the Phenotype table right now. If not, then one may have to operate on the table and merge duplicate entries...

gedankenstuecke commented 8 years ago

@vivekiitkgp I think there were like 2-3 duplicate phenotypes which would have to be manually dealt with before. Thanks for the work on it so far!

tsujigiri commented 8 years ago

I'd prefer that we deal with it through a reviewable and testable migration over fiddeling around with it manually. ;)

gedankenstuecke commented 8 years ago

Fair enough :p

raivivek commented 8 years ago

@tsujigiri On it!