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

docs: Add 2 chains to feature matrix: fuse & blast #979

Draft
wants to merge 2 commits into
base: main
Choose a base branch
from

Conversation

p-diogo
Copy link
Contributor

@p-diogo p-diogo commented Aug 5, 2024

Adds full protocol support for moonbeam, fuse, and blast data sources.

The Council will be reviewing the CIP reports for these chains this Thursday.

Next steps:

  • Create a GGP referencing this PR (Council approving full protocol support for these 3 chains)
  • Wait for the Council to vote
  • Add GGP to the feature matrix
  • Merge PR

EDIT: removed moonbeam from the list.

@abourget
Copy link

abourget commented Aug 8, 2024

As discussed recently with @mangas .. there's an issue with enabling Substreams-powered Subgraphs for names like "moonbeam" is tricky because those chains do not have Extended traces instrumentation..

This creates a problem of data provenance (the source backing a subgraph) needs to be holistically defined, and if we go from Light blocks to Extended blocks, all under the "moonbeam" network name, things will break determinism.

The graph-node is able to abstract from one to the other (offers triggers from both sources), but Substreams get fed the big raw blocks, and those need to be deterministic.

I think it's not safe to enable SpS without defining the exact meaning of moonbeam, and then be ready for, in the future, support moonbeam+extended (and consider moonbeam to always be light blocks).

@p-diogo
Copy link
Contributor Author

p-diogo commented Aug 14, 2024

As we're still debating how to best handle different block types (base or extended, related to @abourget message above), I'm removing moonbeam so we can proceed with the other non-contentious chains.

@p-diogo p-diogo changed the title docs: Add 3 chains to feature matrix: moonbeam, fuse, blast docs: Add 3 chains to feature matrix: fuse & blast Aug 14, 2024
@p-diogo p-diogo changed the title docs: Add 3 chains to feature matrix: fuse & blast docs: Add 2 chains to feature matrix: fuse & blast Aug 14, 2024
dwerner pushed a commit that referenced this pull request Aug 19, 2024
- docs: Add to feature matrix:s
- add moonbeam, fuse and blast to chain id lookups
- removed changes to the feature matrix. that should be handled in #979
- added missing mappings to Caip2ByChainID
dwerner pushed a commit that referenced this pull request Aug 19, 2024
- docs: Add to feature matrix:s
- add moonbeam, fuse and blast to chain id lookups
- removed changes to the feature matrix. that should be handled in #979
- added missing mappings to Caip2ByChainID
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