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

Preview: CHEM are ignored without any messages or incorrect message while we save sequence with CHEM to IDT #4639

Closed
NadezhdaPeskun opened this issue May 17, 2024 · 2 comments

Comments

@NadezhdaPeskun
Copy link
Collaborator

NadezhdaPeskun commented May 17, 2024

Steps to Reproduce
Scenario 1

  1. Switch to Macro
  2. Open KET file with CHEM
    ketcher (middle Chem).zip
  3. try to save to IDT
    image

Scenario 2

  1. Open KET file with only CHEM
    ketcher (only Chems).zip

  2. try to save to IDT >> empty IDT file is created
    ketcher (Empty from only Chems).zip

Empty IDT with Chem

Scenario 3

  1. Open KET file with only CHEM
    ketcher (Chem first).zip

  2. try to save to IDT >> empty IDT file is created
    Chem-Sugar

Actual behavior
CHEM are ignored without any messages or incorrect error message while Chem linked to Sugar.

Expected behavior
According BA solution:
We should have message for CHEM as for Peptides if Chem is not in IDT library
"Cannot save molecule in IDT format - monomer(s)… doesn’t have IDT code."

Win11
Chrome: Version 123.0.6312.123 (Official Build) (64-bit)

Ketcher
[Version 2.22-rc.1] CI
[Indigo Toolkit] CI
Version 1.21.0-rc.1

@NadezhdaPeskun NadezhdaPeskun added this to the Ketcher 2.22.0-rc.1 milestone May 17, 2024
@NadezhdaPeskun NadezhdaPeskun changed the title Preview: CHEM are ignored without any messages while we save sequence with CHEM to IDT Preview: CHEM are ignored without any messages or incorrect message while we save sequence with CHEM to IDT May 17, 2024
@NadezhdaPeskun
Copy link
Collaborator Author

Should be moved to Indigo/

@NadezhdaPeskun
Copy link
Collaborator Author

NadezhdaPeskun commented May 20, 2024

Moved to Indigo:
epam/Indigo#1958

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

1 participant