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.

Sign up for GitHub

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

Jump to bottom

Release 2.0.1 #43

Merged
merged 54 commits into from
Aug 2, 2021
Merged

Release 2.0.1 #43

merged 54 commits into from
Aug 2, 2021

Conversation

ctucker3
Copy link
Contributor

No description provided.

Chris T and others added 30 commits September 16, 2020 10:51
add germplasm model
af5bd10
get all germplasm get specific germplasm
2989531
add files to gitignore
8985a87
POST germplasm + tests
afa43fb
add PUT germplasm + tests
879253b
uncommented previously broken tests
1da907b
pull request changes
faf85d5
additional fixes for pull request review
44a1902
missed fix pull request review
1e4a808
replace null checks with lombok nonnull check
7250e6e
change enum identifiers to descriptive names
c6a77b5
Merge pull request #17 from Breeding-Insight/feature/BI-272
1befb39 
[BI-269, BI-270, BI-271, BI-272] - BrAPI Germplasm Endpoints
@timparsons
Merge pull request #19 from Breeding-Insight/master
3d805b4 
Added release drafter actions
@nickpalladino
Added create locations api and tests
205263b
@nickpalladino
Added get locations api and tests
ca38c30
@nickpalladino
Added update locations api and tests
8a1bbcc
@nickpalladino
Removed dataType query parameters
ae7706b
@nickpalladino
Removed / and end of test url that was resuling in //
d0a3b9c
@BrapiCoordinatorSelby
Selby went rogue
dd0dc9a
@BrapiCoordinatorSelby
everything compiles MFers
fd248c0
@BrapiCoordinatorSelby
all da class say da BrAPI on dem
a7b1827
@timparsons
Refactoring class structure
2b66ba2 
- Created request and response packages with modules
- Merged classes that extended *NewRequest classes as that made for a confusing data model and API, and caused typing issues when using the fluent setters.
@timparsons
Sanity check and cleanup
5faf92a 
- Updating BrAPIClient to check that path ends with /brapi/v2
- removing swagger-generated-client from src/main/resources
@BrapiCoordinatorSelby
stashing
3ff96ad
@BrapiCoordinatorSelby
Merge branch 'swagger-gen' of https://github.com/Breeding-Insight/brapi…
6bdd872 
… into swagger-gen
@BrapiCoordinatorSelby
JUnits are technically passing, not thoroughly testing, but they are …
064d462 
…passing
@BrapiCoordinatorSelby
minor junit cleanup
4b3926d
@BrapiCoordinatorSelby
enums share an interface and all unit tests pass
141b030
@timparsons
Merge branch 'develop' into swagger-gen
620cb50
@timparsons
Merge pull request #25 from Breeding-Insight/swagger-gen
64a0760
@BrapiCoordinatorSelby
fixes when using in lib in Field Book
166973a
@BrapiCoordinatorSelby
cleaned up formatting
f6ef6e0
@BrapiCoordinatorSelby
Merge pull request #28 from Breeding-Insight/fieldbook_fixes
3f4cd1e 
fixes when using in lib in Field Book
@BrapiCoordinatorSelby
handle all exceptions from response
cdd76b9
@timparsons
Merge pull request #29 from Breeding-Insight/fieldbook_fixes
5fc6a7a 
[BI-761] handle all exceptions from response
fix observation level and add observation unit names to observation u…
b8347da 
…nit search
implement search pair return for observation units
3f6bf59
add pair return type for all POST search endpoints
ea97421
fix search endpoint tests
6b84c67
fix search results return type
dd34f5f
add pair result for get search results endpoints
9f7048e
Merge pull request #35 from Breeding-Insight/bug/BI-783
6afe768 
[BI-783,BI-781] - Search response implementation and other bug fixes
@BrapiCoordinatorSelby
fix for issue #32
bfd4662
@BrapiCoordinatorSelby
Merge branch 'develop' into issue-32
22a819f
@BrapiCoordinatorSelby
fix tests
695fdad
@timparsons
Merge pull request #36 from Breeding-Insight/issue-32
4e8631d 
fix for issue #32
@nickpalladino
forgot fixes to search response
d51b16a
@nickpalladino
Fix tests
b0eee49
@timparsons
Merge pull request #38 from Breeding-Insight/bug/BI-793
39a5838 
BI-793 Fix brapi client test errors
@nickpalladino
Add github action to run tests on PRs
a567d0c
make additional info support any object as value for brapi variables
0e3c384
add additional info to all objects
bb2be6e
Merge pull request #40 from Breeding-Insight/feature/BI-998
aebd7a9 
[BI-998] - Make additional info support any object as value for brapi variables
update release version
b38bf5e
timparsons
timparsons approved these changes Jul 30, 2021
View reviewed changes
Copy link
Member

@timparsons timparsons left a comment

Choose a reason for hiding this comment

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

Approved pending figuring out why the build is failing

@ctucker3 ctucker3 merged commit 8742b46 into master Aug 2, 2021
@ctucker3 ctucker3 deleted the release-2.0.1 branch August 2, 2021 16:31
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@nickpalladino nickpalladino nickpalladino approved these changes

@timparsons timparsons timparsons approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
4 participants
@ctucker3 @timparsons @nickpalladino @BrapiCoordinatorSelby