Conversation
| "Variety Release Trial, Clonal Evaluation, Advanced Yield Trial, crossing_trial, " + | ||
| "grafting_trial, pollinating_trial, phenotyping_trial, genotyping_trial, " + | ||
| "Seed Multiplication, Screen House, crossing_block_trial, Specialty Trial, etc.") | ||
| private String studyType; |
There was a problem hiding this comment.
I would remove misc_trial from the list. That's an internal breedbase thing.
There was a problem hiding this comment.
ya, I wasn't sure about that since it's showing up in the trial design as an option in the breedbase GUI. I grabbed all the terms from the db and looks like there's a few extra compared to the GUI.
There was a problem hiding this comment.
Let me know if you still think I should remove it.
There was a problem hiding this comment.
Oh I see. Yeah I had to put in a trial type of misc_trial whose cvterm value is recorded as the type the user passes in. In brapi it returns whatever the user enters and doesn't show misc_trial. So I guess we could show it, I just wonder if it is useful to anyone. I guess these help texts will all need a pass through by Shawn to clean them up anyway, so you can probably just leave it for now.
There was a problem hiding this comment.
ok sounds good thanks.
Importer add studyType to Study
3 participants
Breedbase
POST /studiesrequires the studyType field to be set so adding it as a required field to the importer for the Study object.