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

Deleting an upstream component should delete the downstream references to it #9315

Closed
AdRiley opened this issue Mar 7, 2024 · 2 comments · Fixed by #9659
Closed

Deleting an upstream component should delete the downstream references to it #9315

AdRiley opened this issue Mar 7, 2024 · 2 comments · Fixed by #9659
Assignees
Labels
--bug Type: bug -gui d-intermediate Difficulty: some prior knowledge required p-medium Should be completed in the next few sprints
Milestone

Comments

@AdRiley
Copy link
Member

AdRiley commented Mar 7, 2024

Build a simple workflow

image

and delete a component that connects downstream.

Expected behaviour:

The downstream references should be replaced with _ that can then be reconnected to a new output

image

Actual behaviour:

A reference to the now deleted component is left behind:

image

@AdRiley AdRiley added --bug Type: bug -gui labels Mar 7, 2024
@AdRiley AdRiley added this to the Beta Release milestone Mar 7, 2024
@github-project-automation github-project-automation bot moved this to ❓New in Issues Board Mar 7, 2024
@farmaazon
Copy link
Contributor

I guess we should just do an act of reconnecting everything, as we did in old GUI.

@farmaazon farmaazon moved this from ❓New to 📤 Backlog in Issues Board Mar 7, 2024
@farmaazon farmaazon added d-intermediate Difficulty: some prior knowledge required p-medium Should be completed in the next few sprints labels Mar 7, 2024
@Frizi Frizi self-assigned this Mar 25, 2024
@Frizi Frizi moved this from 📤 Backlog to 🔧 Implementation in Issues Board Apr 9, 2024
@Frizi Frizi moved this from 🔧 Implementation to 👁️ Code review in Issues Board Apr 9, 2024
@enso-bot
Copy link

enso-bot bot commented Apr 11, 2024

Paweł Grabarz reports a new STANDUP for the provided date (2024-04-09):

Progress: Implemented edge cleanup on node deletion. It should be finished by 2024-04-09.

@farmaazon farmaazon moved this from 👁️ Code review to 🟢 Accepted in Issues Board Apr 11, 2024
@farmaazon farmaazon moved this from 🟢 Accepted to 🗄️ Archived in Issues Board Apr 16, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
--bug Type: bug -gui d-intermediate Difficulty: some prior knowledge required p-medium Should be completed in the next few sprints
Projects
Archived in project
Development

Successfully merging a pull request may close this issue.

3 participants