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

typos and recommendation #9

Merged
merged 1 commit into from
Mar 18, 2021
Merged

typos and recommendation #9

merged 1 commit into from
Mar 18, 2021

Conversation

baartch
Copy link
Contributor

@baartch baartch commented Mar 17, 2021

No description provided.

@@ -8,8 +8,8 @@

### Properties

- [#app](API.md##app)
Copy link
Contributor Author

Choose a reason for hiding this comment

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

I assume this file is autogenerated. Am I right? So this error will probably popup again.

@@ -6,7 +6,7 @@ Currently there are Subgraphs deployed on the [xdai](https://thegraph.com/explor

## Queries

Below there are a few ways to show how to query the Subgraph for data. The queries show most of the information that is queryable, but there are many other filtering options that can be used, just check out the [querying api](https://thegraph.com/docs/graphql-api). These queries can be used locally or in The Graph Explorer playground.
Copy link
Contributor Author

Choose a reason for hiding this comment

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

This is just a recommendation. I hope I got this right.

Copy link
Member

Choose a reason for hiding this comment

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

It's great!

Copy link
Member

@PJColombo PJColombo left a comment

Choose a reason for hiding this comment

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

LGTM!

@PJColombo PJColombo merged commit 59b35f1 into CommonsSwarm:main Mar 18, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants