-
Notifications
You must be signed in to change notification settings - Fork 1
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Feature/BI-750 Create an interface to list all studies that exist for a program #94
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
It looks like code is there to display experiments, but a route only exists for trials. Was this intentional?
src/breeding-insight/model/Trial.ts
Outdated
// import {ProgramObservationLevel} from "@/breeding-insight/model/ProgramObservationLevel"; | ||
// import {Method} from "@/breeding-insight/model/Method"; | ||
// import {Scale} from "@/breeding-insight/model/Scale"; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Are these needed for anything?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
nope, removed.
src/breeding-insight/model/Study.ts
Outdated
// import {ProgramObservationLevel} from "@/breeding-insight/model/ProgramObservationLevel"; | ||
// import {Method} from "@/breeding-insight/model/Method"; | ||
// import {Scale} from "@/breeding-insight/model/Scale"; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Are these needed for anything?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
removed
} | ||
|
||
try { | ||
if(!programId) throw new Error('mnissing or invalid program id'); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
"missing" is misspelled
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
thanks! fixed.
data: T | ||
--> | ||
<template v-slot:side-panel="{tableRow}"> | ||
<TraitDetailPanel |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Is this suppose to be the StudyDetailPanel
?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
yes. changed
- limitations under the License. | ||
--> | ||
|
||
<template> |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This appears to be a clone of the TraitDetailPanel
, I assume this will be properly implemented with BI-751?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
correct.
v-on:show-error-notification="$emit('show-error-notification', $event)" | ||
> | ||
<template v-slot="validations"> | ||
<BaseTraitForm |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Should this be BaseStudyForm
?
<template> | ||
<div class="columns is-multiline"> | ||
<div class="column" v-bind:class="{'is-full': editFormat}"> | ||
<BasicInputField |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I assume this will be updated to be specific to studies in BI-751?
ProgramUsersTable | ||
} | ||
}) | ||
export default class ProgramUserManagement extends ProgramsBase { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Class name doesn't match file name
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Per our conversation, experiments has been replaced with studies. All studies are now listed under "Trials and Studies", and the studies for a single trial can be listed by clicking on the name of a trial in the trial list.
TrialsTable | ||
} | ||
}) | ||
export default class ProgramLocationManagement extends TrialsBase { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Class name doesn't match file name
]) | ||
} | ||
}) | ||
export default class ProgramManagement extends ProgramsBase { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Class name doesn't match file name
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks good! Left a comment to remove commented out code, but approving otherwise.
//import TrialsAndExperiments from "@/views/trials-and-studies/TrialsAndExperiments.vue"; | ||
import TrialsAndStudies from "@/views/trials-and-studies/TrialsAndStudies.vue"; | ||
//import Experiments from '@/views/trials-and-studies/Experiments.vue'; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Can the commented out lines be removed?
Models, DAO, and Services created for Trials and Studies.
Route, views and component tables created for Trials and Studies.