-
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
[BI-1366] Subscribe Ontology #216
Conversation
bb7a10a
to
288ad3e
Compare
* limitations under the License. | ||
*/ | ||
|
||
export class SubscribedProgram { |
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 this be renamed to SubscribedOntology
?
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.
renamed along with SharedOntology and SharedOntologyRequest
<p>You cannot subscribe to another programs ontology when you have shared your own program's ontology.</p> | ||
</template> | ||
<template v-else-if="hasTraits"> | ||
<p>This program contains traits and so cannot subscribe to another program's ontology.</p> |
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.
wording suggestion: change "...and so cannot..." to "...and therefore cannot..."
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.
changed
c0213b9
to
53f913e
Compare
Description
Story: https://breedinginsight.atlassian.net/jira/software/c/projects/BI/boards/1?modal=detail&selectedIssue=BI-1366
Added the ability for a program to subscribe to an ontology. Add checks and UI indicators for the following error conditions:
Dependencies
biapi: BI-1366
Testing
observation
table for the brapi server.Checklist: