Skip to content

[BI-2971] - Improve default sample submission sorting - #542

Merged
humsika merged 1 commit into
developfrom
feature/BI-2971
Aug 24, 2026
Merged

[BI-2971] - Improve default sample submission sorting#542
humsika merged 1 commit into
developfrom
feature/BI-2971

Conversation

@HMS17

@HMS17 HMS17 commented Aug 20, 2026

Copy link
Copy Markdown
Contributor

Description

Story: BI-2971 - Improve default sample submission sorting

Change to SampleSubmissionService.java: generateDArTFile to sort submissions samples for export by plate > column > row.

Dependencies

bi-web: feature/BI-2971

Testing

Navigate to submission details for a sample submission
Click on Manage Submission > Generate and Download DArT file
Check that rows in file are sorted by plate > column > row

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 create/modified unit and/or integration tests to cover this change or tests are not applicable
  • I have commented my code, particularly in hard-to-understand areas
  • I have either updated the source of truth or arranged for update with product owner if needed https://breedinginsight.atlassian.net/wiki/spaces/BI/pages/1559953409/Source+of+Truth

@HMS17
HMS17 marked this pull request as ready for review August 20, 2026 22:41
@HMS17
HMS17 requested review from a team and humsika and removed request for a team August 20, 2026 22:42

@humsika humsika left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Looks good! I also validated the changes manually, and everything is working as expected.

@humsika
humsika merged commit 2727862 into develop Aug 24, 2026
2 of 4 checks passed
@humsika
humsika deleted the feature/BI-2971 branch August 24, 2026 13:16
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.

2 participants