Skip to content

[BI-2067] - Merge FieldBook BrAPI program endpoint with Mr.Bean work#410

Merged
HMS17 merged 14 commits intorelease/1.0from
feature/BI-2067
Oct 8, 2024
Merged

[BI-2067] - Merge FieldBook BrAPI program endpoint with Mr.Bean work#410
HMS17 merged 14 commits intorelease/1.0from
feature/BI-2067

Conversation

@HMS17
Copy link
Contributor

@HMS17 HMS17 commented Oct 8, 2024

Description

Story: _BI-2067 - Merge Tim's FieldBook BrAPI program endpoint with Mr.Bean work

Copy of #407 to merge into release

  • Renamed org.breedinginsight.brapi.v2.CropController to org.breedinginsight.brapi.v2.BrAPICropController
  • Removed org.breedinginsight.brapi.v2.ProgramController
  • Merged BrAPIProgramsController::programsGet and ProgramController::getPrograms methods

Dependencies

n/a

Testing

Test that Mr. Bean and Fieldbook functionality related to programs endpoint still works as expected

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 marked this pull request as ready for review October 8, 2024 18:46
@HMS17 HMS17 requested review from mlm483 and nickpalladino October 8, 2024 18:46
@HMS17 HMS17 merged commit 0e826f2 into release/1.0 Oct 8, 2024
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.

4 participants