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

Creating a subgraph of a group causing a crash #418

Open
mck opened this issue Jul 2, 2024 · 3 comments · May be fixed by #455
Open

Creating a subgraph of a group causing a crash #418

mck opened this issue Jul 2, 2024 · 3 comments · May be fixed by #455
Assignees

Comments

@mck
Copy link
Collaborator

mck commented Jul 2, 2024

When attempting to create a subgraph of an existing group the app will crash. This appears due to the fact that the group is not copied across to the subgraph

Image

Image

@mck mck converted this from a draft issue Jul 2, 2024
@mck mck pinned this issue Jul 2, 2024
@mck
Copy link
Collaborator Author

mck commented Jul 9, 2024

lets disable the group function for now since they are also not saved, we can introduce them to a later point

@georgebuciuman georgebuciuman self-assigned this Jul 9, 2024
@georgebuciuman georgebuciuman moved this from 📨 Ready for work to 🚧 In Progress in Graph Engine Jul 9, 2024
@georgebuciuman georgebuciuman moved this from 🚧 In Progress to 👀 Review in Graph Engine Jul 9, 2024
@SorsOps
Copy link
Member

SorsOps commented Jul 9, 2024

No we should be saving the group in the graph

@georgebuciuman georgebuciuman moved this from 👀 Review to 🚧 In Progress in Graph Engine Jul 16, 2024
@georgebuciuman georgebuciuman linked a pull request Jul 18, 2024 that will close this issue
@georgebuciuman georgebuciuman moved this from 🚧 In Progress to 👀 Review in Graph Engine Jul 18, 2024
@SorsOps
Copy link
Member

SorsOps commented Jul 31, 2024

Related #285

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

Successfully merging a pull request may close this issue.

3 participants