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

Update 0054-timeline-aggregation-protocol.md #42

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

tkornuta-semiotic
Copy link
Contributor

Removed references to Scalar TAP to reflect the real state of the solution that we are putting to production.

Removed references to Scalar TAP to reflect the real state of the solution that we are putting to production.
@p-diogo
Copy link

p-diogo commented Aug 7, 2024

@tkornuta-semiotic, I don't see any reason for this PR to remain open. Can I review it quickly and merge it?


More on the prior art can be found in `Rationale and Alternatives` section
More info about Scalar can be found in [The Graph Foundation unveils Scalar: a microtransaction for every query](https://thegraph.com/blog/scalar/) blog post.
More on the prior art can be found in `Rationale and Alternatives` section.
Copy link
Contributor

Choose a reason for hiding this comment

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

It is likely there are other examples, and this was not even a proposed change of this PR, however worth noting we can nicely link such references to other sections.

Suggested change
More on the prior art can be found in `Rationale and Alternatives` section.
More on the prior art can be found in [Rationale and Alternatives](#rationale-and-alternatives) section.

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.

3 participants