Skip to content

[BI-1363] Share Ontology #166

Merged
ctucker3 merged 9 commits intodevelopfrom
feature/BI-1363
Apr 5, 2022
Merged

[BI-1363] Share Ontology #166
ctucker3 merged 9 commits intodevelopfrom
feature/BI-1363

Conversation

@ctucker3
Copy link
Contributor

Description

Story: https://breedinginsight.atlassian.net/jira/software/c/projects/BI/boards/1?modal=detail&selectedIssue=BI-1363

  • Added an OntologyController for sharing, revoke, and retrieving shared ontologies.
  • Added tests
  • Scoped out some TODOs where dependencies prevented me from working further

Dependencies

biweb: BI-1363

Testing

  1. Create multiple programs with the same species
  2. Create an additional program of a different species
  3. Go to the Program Management -> Configuration page
  4. Make sure the programs with same species are able to be shared and program with different species is not.
  5. Share with a program or two
  6. Modal should automatically close and the ontologies should be listed under 'Shared Ontology' now
  7. The shared programs should have a (Not Accepted) next to them (accepting share not in this card)
  8. Revoke the share from a program or two and make sure those are removed from the list of shared programs.

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>

@davedrp
Copy link
Contributor

davedrp commented Mar 30, 2022

passed test

@ctucker3 ctucker3 merged commit e0c5729 into develop Apr 5, 2022
@ctucker3 ctucker3 deleted the feature/BI-1363 branch April 5, 2022 18:25
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