Skip to content

Issue 52657: LKSM: We shouldn't allow creating sample names that differ only in case#1828

Merged
XingY merged 10 commits intodevelopfrom
fb_sampleNameCase
Jul 10, 2025
Merged

Issue 52657: LKSM: We shouldn't allow creating sample names that differ only in case#1828
XingY merged 10 commits intodevelopfrom
fb_sampleNameCase

Conversation

@XingY
Copy link
Contributor

@XingY XingY commented Jul 8, 2025

Rationale

In order to compute the list of names that are duplicate for the editable grid, toLowerCase is called on the raw name to build a map. This altered key is then used in the duplicate error message, which is confusing since it might not match user provided name.

Related Pull Requests

Changes

  • Use original case in duplicate name error message

XingY added 4 commits July 8, 2025 09:49
# Conflicts:
#	packages/components/package-lock.json
#	packages/components/package.json
#	packages/components/releaseNotes/components.md
XingY added 4 commits July 9, 2025 11:58
# Conflicts:
#	packages/components/package-lock.json
#	packages/components/package.json
#	packages/components/releaseNotes/components.md
# Conflicts:
#	biologics/package-lock.json
#	biologics/package.json
#	inventory/package-lock.json
#	inventory/package.json
#	sampleManagement/package-lock.json
#	sampleManagement/package.json
@XingY XingY merged commit 8634056 into develop Jul 10, 2025
3 checks passed
@XingY XingY deleted the fb_sampleNameCase branch July 10, 2025 22:07
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.

2 participants