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

Added new page for Data Indexers under pages/builders/tools/monitor #1148

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

Conversation

alinobrasil
Copy link

Description

This adds a new page for "data indexers" under pages/builders/tools/monitor.
It includes a short guide on The Graph.
Other providers can easily contribute to the same page.

Tests

Sucessfully ran the following:

  • pnpm fix
  • pnpm spellcheck:lint

Was able to run locally with pnpm dev

@alinobrasil alinobrasil requested a review from a team as a code owner November 28, 2024 18:22
Copy link

netlify bot commented Nov 28, 2024

Deploy Preview for docs-optimism ready!

Name Link
🔨 Latest commit 88c6ea0
🔍 Latest deploy log https://app.netlify.com/sites/docs-optimism/deploys/6748b4fd43568e00083d5ed9
😎 Deploy Preview https://deploy-preview-1148--docs-optimism.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site configuration.

Copy link
Contributor

coderabbitai bot commented Nov 28, 2024

📝 Walkthrough

Walkthrough

The changes in this pull request involve the addition of a new card titled "Data Indexers" to the existing <Cards> component in the pages/builders/tools/monitor.mdx file. This card links to a new resource at the path /builders/tools/monitor/data-indexers, which expands the documentation related to analytics tools and accessing pre-Regensis history. Additionally, a new file named data-indexers.mdx has been created, providing a detailed guide on data indexers for decentralized applications (dApps) that interact with blockchain data. This document covers various indexing providers, focusing on "The Graph," detailing its functionalities, including decentralized indexing and GraphQL queries. It also mentions a forthcoming section on Software as a Service (SaaS) providers. Furthermore, the words.txt file has undergone modifications, with several new terms added and others removed, reflecting updates to the vocabulary used in the documentation.

Possibly related PRs

Suggested reviewers

  • sbvegan
  • krofax

Thank you for using CodeRabbit. We offer it for free to the OSS community and would appreciate your support in helping us grow. If you find it useful, would you consider giving us a shout-out on your favorite social media?

❤️ Share
🪧 Tips

Chat

There are 3 ways to chat with CodeRabbit:

  • Review comments: Directly reply to a review comment made by CodeRabbit. Example:
    • I pushed a fix in commit <commit_id>, please review it.
    • Generate unit testing code for this file.
    • Open a follow-up GitHub issue for this discussion.
  • Files and specific lines of code (under the "Files changed" tab): Tag @coderabbitai in a new review comment at the desired location with your query. Examples:
    • @coderabbitai generate unit testing code for this file.
    • @coderabbitai modularize this function.
  • PR comments: Tag @coderabbitai in a new PR comment to ask questions about the PR branch. For the best results, please provide a very specific query, as very limited context is provided in this mode. Examples:
    • @coderabbitai gather interesting stats about this repository and render them as a table. Additionally, render a pie chart showing the language distribution in the codebase.
    • @coderabbitai read src/utils.ts and generate unit testing code.
    • @coderabbitai read the files in the src/scheduler package and generate a class diagram using mermaid and a README in the markdown format.
    • @coderabbitai help me debug CodeRabbit configuration file.

Note: Be mindful of the bot's finite context window. It's strongly recommended to break down tasks such as reading entire modules into smaller chunks. For a focused discussion, use review comments to chat about specific files and their changes, instead of using the PR comments.

CodeRabbit Commands (Invoked using PR comments)

  • @coderabbitai pause to pause the reviews on a PR.
  • @coderabbitai resume to resume the paused reviews.
  • @coderabbitai review to trigger an incremental review. This is useful when automatic reviews are disabled for the repository.
  • @coderabbitai full review to do a full review from scratch and review all the files again.
  • @coderabbitai summary to regenerate the summary of the PR.
  • @coderabbitai resolve resolve all the CodeRabbit review comments.
  • @coderabbitai configuration to show the current CodeRabbit configuration for the repository.
  • @coderabbitai help to get help.

Other keywords and placeholders

  • Add @coderabbitai ignore anywhere in the PR description to prevent this PR from being reviewed.
  • Add @coderabbitai summary to generate the high-level summary at a specific location in the PR description.
  • Add @coderabbitai anywhere in the PR title to generate the title automatically.

Documentation and Community

  • Visit our Documentation for detailed information on how to use CodeRabbit.
  • Join our Discord Community to get help, request features, and share feedback.
  • Follow us on X/Twitter for updates and announcements.

Copy link
Contributor

@coderabbitai coderabbitai bot left a comment

Choose a reason for hiding this comment

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

Actionable comments posted: 0

🧹 Outside diff range and nitpick comments (8)
pages/builders/tools/monitor.mdx (4)

Line range hint 1-7: Enhance the description metadata for clarity and specificity

The current description is repetitive and vague. Consider revising it to be more specific about the monitoring capabilities and tools covered in this section.

  description: >-
-   Learn about monitor in the Optimism ecosystem. This guide provides detailed
-   information and resources about monitor.
+   Learn about monitoring tools and analytics in the Optimism ecosystem. This guide provides
+   detailed information about blockchain analytics, historical data access, and data indexing.

Line range hint 11-12: Fix grammar and capitalization in the introduction

The introductory text contains grammar issues and improper capitalization of proper nouns.

- This section provides information on analytics tools and accessing pre regenesis history. You'll find guide, tool to help you understand and work with these topics.
+ This section provides information on analytics tools and accessing pre-Regenesis history. You'll find guides and tools to help you understand and work with these topics.

Line range hint 14-16: Standardize card titles and fix hyphenation

The card titles need consistent capitalization and proper hyphenation.

- <Card title="Analytics tools" href="/builders/tools/monitor/analytics-tools" />
- <Card title="Accessing pre Regenesis history" href="/builders/tools/monitor/regenesis-history" />
+ <Card title="Analytics Tools" href="/builders/tools/monitor/analytics-tools" />
+ <Card title="Accessing Pre-Regenesis History" href="/builders/tools/monitor/regenesis-history" />

19-20: Remove extra newline and standardize new card title

There's an unnecessary blank line before the new card, and its title should follow the same capitalization pattern.

-
-  <Card title="Data Indexers" href="/builders/tools/monitor/data-indexers" />
+  <Card title="Data Indexers" href="/builders/tools/monitor/data-indexers" />
pages/builders/tools/monitor/data-indexers.mdx (4)

7-10: Maintain consistent capitalization and avoid personal pronouns.

For better consistency:

  1. Standardize "dApp" vs "dapp" usage throughout the document
  2. Replace "your dApp" with "an application" to maintain neutral tone
-Data indexing is essential for enabling your dApp to efficiently read blockchain data.
+Data indexing is essential for enabling applications to efficiently read blockchain data.

11-12: Remove personal pronoun for consistency.

-Below is a list of indexing providers that can help you index data for your smart contract on Optimism.
+Below is a list of indexing providers that can help index data for smart contracts on Optimism.

21-23: Avoid using italics for emphasis in bullet points.

Per the style guidelines, avoid using italics for emphasis. The bullet points should be formatted as regular text.

-*   Decentralized Indexing: Improves redundancy and query speed. Lets you eliminate any single point of failure.
-*   GraphQL Queries: Provides a powerful GraphQL interface for querying indexed data, making data retrieval super simple.
-*   Customization: Define your own logic for transforming & storing blockchain data. Reuse subgraphs published by other developers on The Graph Network.
+*   Decentralized indexing: Improves redundancy and query speed. Lets you eliminate any single point of failure.
+*   GraphQL queries: Provides a powerful GraphQL interface for querying indexed data, making data retrieval super simple.
+*   Customization: Define your own logic for transforming and storing blockchain data. Reuse subgraphs published by other developers on The Graph Network.

32-34: Enhance the placeholder text for the SaaS Providers section.

Consider adding more context about what kind of information will be added in this section.

## SaaS Providers

-(Coming soon)
+This section will provide information about Software as a Service (SaaS) providers that offer blockchain data indexing solutions. Content coming soon.
📜 Review details

Configuration used: .coderabbit.yaml
Review profile: CHILL

📥 Commits

Reviewing files that changed from the base of the PR and between 7710b44 and 88c6ea0.

📒 Files selected for processing (3)
  • pages/builders/tools/monitor.mdx (1 hunks)
  • pages/builders/tools/monitor/data-indexers.mdx (1 hunks)
  • words.txt (7 hunks)
🧰 Additional context used
📓 Path-based instructions (2)
pages/builders/tools/monitor.mdx (1)

Pattern **/*.mdx: "ALWAYS review Markdown content THOROUGHLY with the following criteria:

  • Use proper nouns in place of personal pronouns like 'We' and 'Our' to maintain consistency in communal documentation.
  • Avoid gender-specific language and use the imperative form.
  • Monitor capitalization for emphasis. Avoid using all caps, italics, or bold for emphasis.
  • Ensure proper nouns are capitalized in sentences.
  • Apply the Oxford comma.
  • Use proper title case for buttons, tab names, page names, and links. Sentence case should be used for body content and short phrases, even in links.
  • Use correct spelling and grammar at all times (IMPORTANT).
  • For H1, H2, and H3 headers:
    1. Use sentence case, capitalizing only the first word.
    2. Preserve the capitalization of proper nouns, technical terms, and acronyms as defined in the 'nouns.txt' file located in the root directory of the project.
    3. Do not automatically lowercase words that appear in the 'nouns.txt' file, regardless of their position in the header.
  • Flag any headers that seem to inconsistently apply these rules for manual review.
  • When reviewing capitalization, always refer to the 'nouns.txt' file for the correct capitalization of proper nouns and technical terms specific to the project.
    "
pages/builders/tools/monitor/data-indexers.mdx (1)

Pattern **/*.mdx: "ALWAYS review Markdown content THOROUGHLY with the following criteria:

  • Use proper nouns in place of personal pronouns like 'We' and 'Our' to maintain consistency in communal documentation.
  • Avoid gender-specific language and use the imperative form.
  • Monitor capitalization for emphasis. Avoid using all caps, italics, or bold for emphasis.
  • Ensure proper nouns are capitalized in sentences.
  • Apply the Oxford comma.
  • Use proper title case for buttons, tab names, page names, and links. Sentence case should be used for body content and short phrases, even in links.
  • Use correct spelling and grammar at all times (IMPORTANT).
  • For H1, H2, and H3 headers:
    1. Use sentence case, capitalizing only the first word.
    2. Preserve the capitalization of proper nouns, technical terms, and acronyms as defined in the 'nouns.txt' file located in the root directory of the project.
    3. Do not automatically lowercase words that appear in the 'nouns.txt' file, regardless of their position in the header.
  • Flag any headers that seem to inconsistently apply these rules for manual review.
  • When reviewing capitalization, always refer to the 'nouns.txt' file for the correct capitalization of proper nouns and technical terms specific to the project.
    "
🔇 Additional comments (2)
pages/builders/tools/monitor/data-indexers.mdx (1)

1-5: LGTM! Frontmatter is well-structured.

The metadata is properly formatted with clear title and description.

words.txt (1)

13-13: LGTM! The new terms are relevant additions.

The added terms are appropriate for blockchain documentation and properly maintain alphabetical sorting.

Also applies to: 151-151, 199-199, 294-295, 328-329, 372-372, 400-400

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.

1 participant