Skip to content

[BI-1198] All Experiments Table#201

Merged
HMS17 merged 3 commits intofuture/1.0from
feature/BI-1198-2
Aug 8, 2022
Merged

[BI-1198] All Experiments Table#201
HMS17 merged 3 commits intofuture/1.0from
feature/BI-1198-2

Conversation

@HMS17
Copy link
Contributor

@HMS17 HMS17 commented Aug 1, 2022

Description

Story: BI-1198 - All Experiments Table

Added /trial endpoint for Experiments and Observations table
Added ExperimentController.java, BrAPITrialService.java, and ExperimentQueryMapper.java
Modified BrAPITrialDAO and added getTrials and processExperimentsForDisplay methods

Dependencies

bi-web/BI-1198-2

Testing

see bi-web

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>

@HMS17 HMS17 changed the base branch from develop to future/1.0 August 1, 2022 17:57
@HMS17 HMS17 changed the title Feature/bi 1198 2 [BI-1198] All Experiments Table Aug 1, 2022
@HMS17 HMS17 marked this pull request as ready for review August 1, 2022 18:40
@HMS17 HMS17 requested review from davedrp and nickpalladino August 1, 2022 18:40
@HMS17 HMS17 force-pushed the feature/BI-1198-2 branch from 41add78 to ec6053b Compare August 3, 2022 14:31
@davedrp
Copy link
Contributor

davedrp commented Aug 3, 2022

passed developer testing

@HMS17 HMS17 requested a review from nickpalladino August 4, 2022 18:15
@HMS17 HMS17 merged commit 806ccd7 into future/1.0 Aug 8, 2022
@HMS17 HMS17 deleted the feature/BI-1198-2 branch August 8, 2022 13:47
timparsons pushed a commit that referenced this pull request Sep 15, 2022
timparsons pushed a commit that referenced this pull request Sep 29, 2022
timparsons pushed a commit that referenced this pull request Oct 10, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants