Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

#4853-Search monomer in library by IDT #5373

Merged
merged 13 commits into from
Sep 2, 2024
Merged

Conversation

KristinaKasyanovskaya
Copy link
Collaborator

@KristinaKasyanovskaya KristinaKasyanovskaya commented Aug 27, 2024

Issue => #4853

Updates:

  • User be able to search by IDT aliases in a library using search bar.

  • Single symbol / in a query result in displaying all the monomers that have IDT alias.

  • The symbol after / is the first symbol of the IDT alias.

  • The symbol before the second / is the last symbol of the IDT alias.

  • Searching diPr/ should return a monomer with alias i6diPr, but searching diP/` should not.

  • If there is any symbol after the second slash, there shouldn't be any results.

  • The search is not case sensitive.

Check list

  • unit-tests written
  • e2e-tests written
  • documentation updated
  • PR name follows the pattern #1234 – issue name
  • branch name doesn't contain '#'
  • PR is linked with the issue
  • base branch (master or release/xx) is correct
  • task status changed to "Code review"
  • reviewers are notified about the pull request

@KristinaKasyanovskaya KristinaKasyanovskaya marked this pull request as ready for review August 30, 2024 13:45
@auto-assign auto-assign bot requested a review from Zhirnoff August 30, 2024 13:45
Copy link
Collaborator

@rrodionov91 rrodionov91 left a comment

Choose a reason for hiding this comment

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

Comments will be fixed in future PR

const modifications = item.idtAliases?.modifications;
let transformedIdtText = idtName;

if (item.name?.includes('MOE') && idtName) {
Copy link
Collaborator

Choose a reason for hiding this comment

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

We should not rely on monomer name because we will have other monomers with idt aliases in future

Comment on lines +432 to +446
switch (position) {
case PresetPosition.Library:
transformedIdtText = `${endpoint5}, ${internal}`;
break;
case PresetPosition.ChainStart:
transformedIdtText = endpoint5;
break;
case PresetPosition.ChainMiddle:
transformedIdtText = internal;
break;
case PresetPosition.ChainEnd:
transformedIdtText = endpoint3;
break;
}
}
Copy link
Collaborator

Choose a reason for hiding this comment

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

Position does not play any role in library.

@rrodionov91 rrodionov91 merged commit 34e4788 into master Sep 2, 2024
5 checks passed
@rrodionov91 rrodionov91 deleted the 4853-search-monomer-by-idt branch September 2, 2024 16:59
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.

Search monomer in library by IDT
2 participants