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

Preparing changes of uid syntax* association. And fixed a writing error in the docs. #3002

Merged
merged 4 commits into from
Mar 1, 2019

Conversation

MichelDiz
Copy link
Contributor

@MichelDiz MichelDiz commented Feb 11, 2019

Rectifying changes of uid syntax in favor of #2895

Some predicates need to be revised if they even need the change. According to its purpose (like "rated" is it a list or a single 1 to 1?). As far as I can tell, it is okay.

Awaiting* for review.


This change is Reviewable

…ror in the docs.

Rectifying changes of uid syntax in favor of #2895

Some predicates need to be revised if they even need the change. According to its purpose (like "rated" is it a list or a single 1 to 1?). As far as I can analyze, it is okay.

Awaiting* for review.
Copy link
Contributor

@martinmr martinmr left a comment

Choose a reason for hiding this comment

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

Reviewed 10 of 10 files at r1.
Reviewable status: :shipit: complete! all files reviewed, all discussions resolved

Updated the Query formart in the "queries" section of design-concepts.

Added an example of structure in GraphQL+- Fundamentals.
@danielmai danielmai merged commit 80557ab into dgraph-io:master Mar 1, 2019
@danielmai danielmai mentioned this pull request Mar 2, 2019
dna2github pushed a commit to dna2fork/dgraph that referenced this pull request Jul 19, 2019
…or in the docs. (dgraph-io#3002)

* Fixed a writing error in docs

* Preparing changes of uid syntax * association. And fixed a writing error in the docs.

Rectifying changes of uid syntax in favor of dgraph-io#2895

Some predicates need to be revised if they even need the change. According to its purpose (like "rated" is it a list or a single 1 to 1?). As far as I can analyze, it is okay.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

Successfully merging this pull request may close these issues.

3 participants