Skip to content

BI-1915 - Export Ontology File#310

Merged
davedrp merged 6 commits intodevelopfrom
feature/BI-1915
Nov 18, 2023
Merged

BI-1915 - Export Ontology File#310
davedrp merged 6 commits intodevelopfrom
feature/BI-1915

Conversation

@davedrp
Copy link
Contributor

@davedrp davedrp commented Nov 8, 2023

Description

BI-1915 Export Ontology File

Additional work...

  • The Import file option of the Manage Ontology pull-down is now only clickable for users that have the right to import.
  • Addressed some of the preexisting code 'problems' reported by IntelleJ.

Dependencies

bi-web: feature/BI-1915

Testing

  • Go to Ontology page.
  • Select the Download file option from the Manage Ontology pull-down.
  • Review the downloaded file;
    . it should have a specified filename ( <program name>_ Active_Ontology_<timestamp> ).
    . it should match the ontology template (including tags).
    . it should include all of the active ontologies in the program.
    . it should be in Name order.

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 tested that my code works with both the brapi-java-server and BreedBase
  • 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: <please include a link to TAF run>

@davedrp davedrp requested review from a team, dmeidlin and nickpalladino and removed request for a team November 8, 2023 17:52
@davedrp
Copy link
Contributor Author

davedrp commented Nov 18, 2023

Unit Test Failed. Merging anyway.

  • There is a known bug in the Unit Test.
  • passed unit tests before merging with develop branch.

@davedrp davedrp merged commit 5881a34 into develop Nov 18, 2023
@davedrp davedrp deleted the feature/BI-1915 branch November 18, 2023 22:07
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