Skip to content

fix(GraphQL): Dgraph directive with reverse edge should work smoothly with interfaces (#5911)#5982

Merged
parasssh merged 1 commit intorelease/v20.07from
pawanrawal/cherrypick-5744
Jul 14, 2020
Merged

fix(GraphQL): Dgraph directive with reverse edge should work smoothly with interfaces (#5911)#5982
parasssh merged 1 commit intorelease/v20.07from
pawanrawal/cherrypick-5744

Conversation

@pawanrawal
Copy link
Copy Markdown
Contributor

@pawanrawal pawanrawal commented Jul 14, 2020

Fixes #5744 and GRAPHQL-558

There was another bug where field which map to reverse edges in Dgraph where generated as part of AddTypeInput and TypeRef. They have been excluded now as we can't perform a mutation along them. They would earlier have given an error at runtime while doing mutations in Dgraph about predicate being incorrect because it starts with ~. Apart, from that to fix #5744, we also allow interfaces implemented by the current type in the check.

(cherry picked from commit ecec071)


This change is Reviewable

Docs Preview: Dgraph Preview

… with interfaces (#5911)

Fixes #5744

There was another bug where field which map to reverse edges in Dgraph where generated as part of AddTypeInput and TypeRef. They have been excluded now as we can't perform a mutation along them. They would earlier have given an error at runtime while doing mutations in Dgraph about predicate being incorrect because it starts with ~. Apart, from that to fix #5744, we also allow interfaces implemented by the current type in the che

(cherry picked from commit ecec071)
@github-actions github-actions Bot added the area/graphql Issues related to GraphQL support on Dgraph. label Jul 14, 2020
@parasssh parasssh merged commit fc8104e into release/v20.07 Jul 14, 2020
@pawanrawal pawanrawal deleted the pawanrawal/cherrypick-5744 branch July 31, 2020 08:34
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

area/graphql Issues related to GraphQL support on Dgraph.

Development

Successfully merging this pull request may close these issues.

2 participants