Skip to content
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

Merged
merged 9 commits into from
Apr 19, 2022
Merged

[BI-1366] Subscribe Ontology #216

merged 9 commits into from
Apr 19, 2022

Conversation

ctucker3
Copy link
Contributor

@ctucker3 ctucker3 commented Apr 5, 2022

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:

  1. If program is subscribe to ontology, cannot share their own ontology.
  • Program should not see share option when subscribed to another ontology.
  1. If a program has traits, cannot subscribe to an ontology.
  • The program should not see an option to subscribe and will see an explanation why.
  1. If a program is subscribed to an ontology and has observations for those traits, cannot unsubscribe.
  • Program will not see an option to unsubscribe and a message with explain why.
  • For the sharing program, the checkbox will be greyed out and unclickable for the subscribed program.

Dependencies

biapi: BI-1366

Testing

  • Test the scenarios above.
  • To create an observation, create a trial, study, observation unit, observation (in that order) for the program you want to add an observation to.
  • You can delete an observation by deleting the data in the observation table for the brapi server.

Checklist:

  • I have performed a self-review of my own code
  • I have tested my code and ensured it meets the acceptance criteria of the story
  • I have create/modified unit tests to cover this change
  • I have commented my code, particularly in hard-to-understand areas
  • I have made corresponding changes to documentation
  • I have run TAF: <link to TAF run>

@ctucker3 ctucker3 marked this pull request as ready for review April 6, 2022 15:38
@ctucker3 ctucker3 requested review from a team, timparsons and dmeidlin and removed request for a team April 6, 2022 15:38
* limitations under the License.
*/

export class SubscribedProgram {
Copy link
Member

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?

Copy link
Contributor Author

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>
Copy link
Member

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..."

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

changed

@ctucker3 ctucker3 requested a review from timparsons April 11, 2022 14:27
@ctucker3 ctucker3 merged commit 263abdd into develop Apr 19, 2022
@ctucker3 ctucker3 deleted the feature/BI-1366 branch April 19, 2022 20:36
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants