Skip to content

BI-2359 Unknown error when uploading appended experiment with timestamps downloaded#415

Merged
davedrp merged 2 commits intorelease/1.0from
bug/BI-2359
Oct 29, 2024
Merged

BI-2359 Unknown error when uploading appended experiment with timestamps downloaded#415
davedrp merged 2 commits intorelease/1.0from
bug/BI-2359

Conversation

@davedrp
Copy link
Contributor

@davedrp davedrp commented Oct 23, 2024

Description

BI-2359 Unknown error when uploading appended experiment with timestamps downloaded

Added defensive code to prevent NullPointer Exception

Dependencies

bi-web: release/1.0

Testing

See Acceptance Criteria in story.

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, HMS17 and kr358 and removed request for a team October 23, 2024 17:27
@davedrp davedrp changed the base branch from develop to release/1.0 October 23, 2024 17:34
@davedrp davedrp requested review from mlm483 and removed request for kr358 October 23, 2024 19:58
Copy link
Contributor

@HMS17 HMS17 left a comment

Choose a reason for hiding this comment

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

Passed acceptance criteria!

@davedrp davedrp merged commit bae27e0 into release/1.0 Oct 29, 2024
@davedrp davedrp deleted the bug/BI-2359 branch October 29, 2024 14:39
@mlm483 mlm483 restored the bug/BI-2359 branch November 4, 2024 19:18
@mlm483 mlm483 deleted the bug/BI-2359 branch November 4, 2024 19:24
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