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

add docs for subgraph debug forking #31

Merged
merged 2 commits into from
Feb 10, 2022

Conversation

VIVelev
Copy link
Contributor

@VIVelev VIVelev commented Jan 14, 2022

Migrated over from docs-archived.
Reference: graphprotocol/graph-node#2995

@VIVelev VIVelev requested a review from a team as a code owner January 14, 2022 16:53
@evaporei evaporei self-requested a review January 28, 2022 13:16

Using **subgraph forking** we can essentially eliminate this step. Here is how it looks:

0. Spin-up a local Graph node with the ***appropriate fork-base*** set.
Copy link
Contributor

Choose a reason for hiding this comment

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

Should this list and the one on line 83 start with 0?

I like how computers index things, but I guess humans are weird... 😆

Copy link
Contributor Author

Choose a reason for hiding this comment

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

The idea is that these steps starting with 0 are really "setup" steps and not part of the debug loop.

Yeah... Humans have their quirks, hah.

@evaporei
Copy link
Contributor

I said it on the last time I've reviewed your documentation, but I would like to note it again, I really like the humor in your writing 😄

@evaporei evaporei merged commit 231a5ac into graphprotocol:main Feb 10, 2022
}
```

Oops, how unfortunate, when I deploy my perfect looking subgraph to the [HostedService](https://thegraph.com/hosted-service/) it fails with the *"Gravatar not found!"* error.
Copy link
Member

Choose a reason for hiding this comment

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

@VIVelev Usually, we refer to "Hosted Service" not "HostedService"

Copy link
Contributor

Choose a reason for hiding this comment

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

Fixed here: #121

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.

4 participants