Closed seanderickson closed 9 years ago
Sean, this looks good to me, and yes, we should definitely allow blank values.
Thank you! Katrina
From: Sean Erickson [mailto:notifications@github.com] Sent: Tuesday, December 09, 2014 7:40 PM To: hmsiccbl/screensaver Cc: Rudnicki, Katrina L. Subject: [screensaver] Add "Gender" to the user profile (#150)
controlled vocabulary: [male, female]
@JAS60https://github.com/JAS60 @KatrinaRudnickihttps://github.com/KatrinaRudnicki
— Reply to this email directly or view it on GitHubhttps://github.com/hmsiccbl/screensaver/issues/150.
implemented, please visit https://dev.screensaver.med.harvard.edu/screensaver and verify this functionality.
Hi Sean,
I think this looks good, the only other thing might be to be able to have it as a column option under ‘Screener’ view if we need to do an output of users.
Thank you!
Katrina
From: Sean Erickson [mailto:notifications@github.com] Sent: Wednesday, December 10, 2014 10:46 PM To: hmsiccbl/screensaver Cc: Rudnicki, Katrina L. Subject: Re: [screensaver] Add "Gender" to the user profile (#150)
implemented, please visit https://dev.screensaver.med.harvard.edu/screensaver and verify this functionality.
— Reply to this email directly or view it on GitHubhttps://github.com/hmsiccbl/screensaver/issues/150#issuecomment-66565194.
controlled vocabulary: [male, female]
@JAS60 @KatrinaRudnicki