-
Notifications
You must be signed in to change notification settings - Fork 160
Update math operations to new translation layer #6774
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
Conversation
|
5bd9ba7 to
76ba380
Compare
76ba380 to
4979c75
Compare
packages/graphql/src/translate/queryAST/factory/Operations/DeleteFactory.ts
Show resolved
Hide resolved
| ...mutationSubqueries.map((sq) => | ||
| Cypher.utils.concat(new Cypher.With(nestedContext.target), new Cypher.Call(sq, [nestedContext.target])) | ||
| ) | ||
| ...mutationSubqueries.map((sq) => Cypher.utils.concat(new Cypher.With("*"), new Cypher.Call(sq, "*"))) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
is the "*" instead of nestedContext.target on purpose?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Yes, we need both, the target and the relationship, we can probably refine it further once all the tests are working
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
is a todo worth it?
No description provided.