Skip to content

[BI-2977] - Genotype Results: UI/UX Improvements (w/o delete) - #500

Merged
nickpalladino merged 1 commit into
developfrom
feature/BI-2977
Oct 8, 2026
Merged

nickpalladino merged 1 commit into
developfrom
feature/BI-2977

Conversation

@HMS17

@HMS17 HMS17 commented Oct 8, 2026 •

Copy link
Copy Markdown
Contributor

Description

Story: BI-2977 - Genotype Results: UI/UX Improvements (w/o delete)

Based on QA feedback, this fix changes the url for the genotypes page to end in "/genotypes" instead of "/genotyping" for consistency

Dependencies

bi-api: develop

Testing

Navigate to Genotypes page
Ensure url ends in "/genotypes" rather than "/genotyping"

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 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
  • I have run SiteImprove on pages impacted by changes

@HMS17
HMS17 marked this pull request as ready for review October 8, 2026 13:00
@HMS17
HMS17 requested review from a team, nickpalladino and syarnes-ufl and removed request for a team and syarnes-ufl October 8, 2026 13:00
Comment thread src/router/index.ts
title: 'Genotypes',
layout: layouts.userSideBar
},
component: Genotyping,

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Could also standardize the component name to Genotypes as well but I don't think it's a big deal.

@nickpalladino
nickpalladino merged commit 565b1ed into develop Oct 8, 2026
2 checks passed
@nickpalladino
nickpalladino deleted the feature/BI-2977 branch October 8, 2026 19:22
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