Skip to content
This repository was archived by the owner on Mar 29, 2025. It is now read-only.

Docs(GraphQL): Emphasize that @search annotation is required for filtering on predicates #247

Merged
merged 5 commits into from
Jul 19, 2021

Conversation

catherineluse
Copy link
Contributor

I missed this requirement when reading this page, so I couldn't understand why I got weird errors like "Field "virtualEventUrl" is not defined by type EventFilter." I understood it after getting help in the forums. https://discuss.dgraph.io/t/how-do-you-create-parameterized-graphql-queries/14711 Hopefully this will prevent others from making the same mistake.

…edicates

I missed this requirement when reading this page, so I couldn't understand why I got weird errors like "Field "virtualEventUrl" is not defined by type EventFilter." I understood it after getting help in the forums. https://discuss.dgraph.io/t/how-do-you-create-parameterized-graphql-queries/14711 Hopefully this will prevent others from making the same mistake.
@catherineluse catherineluse requested a review from danielmai as a code owner June 24, 2021 03:51
Copy link
Contributor

@aaroncarey aaroncarey left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Great content! I left a few suggested edits :)

catherineluse and others added 4 commits June 24, 2021 18:58
Co-authored-by: aaroncarey <31550444+aaroncarey@users.noreply.github.com>
Co-authored-by: aaroncarey <31550444+aaroncarey@users.noreply.github.com>
Co-authored-by: aaroncarey <31550444+aaroncarey@users.noreply.github.com>
Co-authored-by: aaroncarey <31550444+aaroncarey@users.noreply.github.com>
Copy link
Contributor

@danielmai danielmai left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks for the PR @catherineluse!

@selectiveduplicate selectiveduplicate merged commit ea8826a into dgraph-io:release/v21.03 Jul 19, 2021
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Development

Successfully merging this pull request may close these issues.

4 participants