Skip to content

LIMS-86: Allow cloning a protein with a dot in the name#494

Merged
NKatti2011 merged 2 commits intoprereleasefrom
fix/LIMS-86/clone-protein-with-dot-in-name
Mar 9, 2023
Merged

LIMS-86: Allow cloning a protein with a dot in the name#494
NKatti2011 merged 2 commits intoprereleasefrom
fix/LIMS-86/clone-protein-with-dot-in-name

Conversation

@ndg63276
Copy link
Collaborator

@ndg63276 ndg63276 commented Mar 9, 2023

Ticket: LIMS-86

  • Add a new validation type, with dots allowed
  • Add validation message for this type and a couple of others
  • Add a couple of hooks for Selenium tests

@ndg63276 ndg63276 requested review from MattPrit and NKatti2011 March 9, 2023 11:40
Copy link
Contributor

@MattPrit MattPrit left a comment

Choose a reason for hiding this comment

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

Changes look okay; I can't think of any reason for not allowing dots in a sample's name.
@John-Holt-Tessella agrees.

@NKatti2011 NKatti2011 merged commit aa7c3eb into prerelease Mar 9, 2023
@John-Holt-Tessella John-Holt-Tessella deleted the fix/LIMS-86/clone-protein-with-dot-in-name branch June 13, 2023 11:38
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