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

Make it work #768

Open
wants to merge 9 commits into
base: main
Choose a base branch
from

Conversation

ArtSuslov
Copy link

No description provided.

@fordN
Copy link
Contributor

fordN commented Sep 6, 2023

Hey @ArtSuslov 👋

Without any comment or description I can only make assumptions about the intent of this PR.

I'm always open to improving usability of the docker containers, etc... but there are a bunch of changes here that I don't see helping the primary users, Graph network indexers.

  • we will not be relying on '@tokene-q/common-ts' instead of @graphrotocol/common-ts, are there differences there that you require?
  • we are hoping to upgrade our docker images to use a newer version of the Node runtime. Version 14 is quite old now and even the version we are using, 16, is no longer technically supported.

I like that you included some instructions for interacting directly with the smart contracts without relying on a browser app.

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.

None yet

3 participants