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

#2742 [Part 1] Refactor R-Group attachment points as entities #2928

Closed
wants to merge 2 commits into from

Conversation

yuleicul
Copy link
Collaborator

@yuleicul yuleicul commented Jul 18, 2023

This PR is one of the small PRs to resolve #2742 for easy review.
This PR is open for reviews but please do not merge it.

What is done in this PR?

rgroup-ap-can

Refactors R-Group attachment point into entity "RGroupAttachmentPoint" and render object "ReRGroupAttachmentPoint", and adds related operations and actions.

How is it refactored?

Ketcher - refactor rgroup attachment point

What is NOT done in this PR?

Two todos and one bug needs to be fixed in other small PRs.

rgroup-ap-cannot-selection

rgroup-ap-cannot-template

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"

@yuleicul yuleicul changed the title #2742 [Part 1 - Refactor] - Attachment point selection edition and deletion #2742 [Part 1] Refactor R-Group attachment points as entities Jul 24, 2023
@yuleicul
Copy link
Collaborator Author

yuleicul commented Aug 1, 2023

Merged by #2978

@yuleicul yuleicul closed this Aug 1, 2023
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.

Attachment point selection edition and deletion
1 participant