Skip to content

remove alchemy banner#761

Merged
nikbhintade merged 1 commit intomainfrom
nb/remove-alchemy-banner
Dec 17, 2025
Merged

remove alchemy banner#761
nikbhintade merged 1 commit intomainfrom
nb/remove-alchemy-banner

Conversation

@nikbhintade
Copy link
Member

@nikbhintade nikbhintade commented Dec 17, 2025

Summary by CodeRabbit

  • Documentation
    • Expanded HyperRPC supported networks documentation with complete network table.
    • Expanded HyperSync supported networks documentation with comprehensive network entries and tier updates.
    • Removed site announcement banner.

✏️ Tip: You can customize this high-level summary in your review settings.

@nikbhintade nikbhintade self-assigned this Dec 17, 2025
@vercel
Copy link

vercel bot commented Dec 17, 2025

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Review Updated (UTC)
envio-docs Ready Ready Preview, Comment Dec 17, 2025 10:51am

@coderabbitai
Copy link
Contributor

coderabbitai bot commented Dec 17, 2025

Walkthrough

Documentation tables for supported networks in HyperRPC and HyperSync were updated with new network entries. Announcement banner configurations were removed from Docusaurus config files.

Changes

Cohort / File(s) Change Summary
Supported Networks Documentation
docs/HyperRPC/hyperrpc-supported-networks.md, docs/HyperSync/hypersync-supported-networks.md
Added comprehensive network tables with entries including Abstract, Arbitrum, Aurora, Avalanche, Base, Berachain, Blast, Celo, Chainweb, Eclipse, Fantom, Fraxtal, Gnosis, Harmony, Linea, Lisk, Manta, Mantle, Moonbeam, Optimism, Polygon, Scroll, Sepolia, Sonic, Taraxa, and others. Updated Base Traces tier designation in HyperSync documentation.
Docusaurus Configuration
docusaurus.config.js, docusaurus.config.llm.js
Removed announcementBar configuration blocks from themeConfig, including banner IDs, HTML content, styling properties (backgroundColor, textColor), and isCloseable flags.

Estimated code review effort

🎯 2 (Simple) | ⏱️ ~8 minutes

  • Documentation changes are primarily table content additions and updates—straightforward to verify for accuracy and formatting consistency
  • Configuration removals are simple deletions with no complex side effects or dependencies to trace
  • Changes are homogeneous in nature (documentation updates and config removals), reducing cognitive load per file

Possibly related PRs

Suggested reviewers

  • moose-code

Poem

🐰 Hop, hop! Tables bloom with networks new,
From Abstract to Taraxa, quite the zoo!
Banners vanish, configs cleaned with care,
Docs shine brighter, floating light as air!

Pre-merge checks and finishing touches

✅ Passed checks (3 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check ✅ Passed The title 'remove alchemy banner' directly matches the main change across multiple config files where the announcementBar configuration is removed.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.
✨ Finishing touches
  • 📝 Generate docstrings
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Post copyable unit tests in a comment
  • Commit unit tests in branch nb/remove-alchemy-banner

📜 Recent review details

Configuration used: defaults

Review profile: CHILL

Plan: Pro

📥 Commits

Reviewing files that changed from the base of the PR and between 6afabc2 and 272c197.

📒 Files selected for processing (4)
  • docs/HyperRPC/hyperrpc-supported-networks.md (1 hunks)
  • docs/HyperSync/hypersync-supported-networks.md (2 hunks)
  • docusaurus.config.js (0 hunks)
  • docusaurus.config.llm.js (0 hunks)
💤 Files with no reviewable changes (2)
  • docusaurus.config.llm.js
  • docusaurus.config.js
🧰 Additional context used
🧠 Learnings (2)
📓 Common learnings
Learnt from: DenhamPreen
Repo: enviodev/docs PR: 673
File: docs/HyperIndex/supported-networks/chainweb-testnet-21.md:14-16
Timestamp: 2025-07-29T14:30:03.677Z
Learning: Chainweb Testnet 20 (ID: 5920) and Chainweb Testnet 21 (ID: 5921) have non-functional HyperSync endpoints that return HTTP 404, indicating these networks are not properly supported and should be removed from documentation.
📚 Learning: 2025-07-29T14:30:03.677Z
Learnt from: DenhamPreen
Repo: enviodev/docs PR: 673
File: docs/HyperIndex/supported-networks/chainweb-testnet-21.md:14-16
Timestamp: 2025-07-29T14:30:03.677Z
Learning: Chainweb Testnet 20 (ID: 5920) and Chainweb Testnet 21 (ID: 5921) have non-functional HyperSync endpoints that return HTTP 404, indicating these networks are not properly supported and should be removed from documentation.

Applied to files:

  • docs/HyperRPC/hyperrpc-supported-networks.md
  • docs/HyperSync/hypersync-supported-networks.md
🪛 markdownlint-cli2 (0.18.1)
docs/HyperSync/hypersync-supported-networks.md

59-59: Bare URL used

(MD034, no-bare-urls)


59-59: Bare URL used

(MD034, no-bare-urls)

🔇 Additional comments (4)
docs/HyperRPC/hyperrpc-supported-networks.md (1)

38-124: Network table looks comprehensive and well-structured.

The 87 network entries are consistently formatted with clear URL patterns for both name-based and ID-based endpoints. This provides good flexibility for users. No issues detected with the table structure or data format.

docs/HyperSync/hypersync-supported-networks.md (3)

46-131: Network and tier table is comprehensive and well-organized.

The 87 network entries with tier indicators provide clear service-level expectations to users. URL patterns are consistent, and tier assignments appear deliberate per-network rather than categorical. Data consistency with HyperRPC documentation is maintained.


59-59: Verify the Base Traces tier downgrade.

Line 59 shows Base Traces (Network ID 8453) downgraded to tier 🪨, whereas Eth Traces (same mainnet, ID 1) retains tier 🏅. This tier difference warrants confirmation: is the reduced support intentional, or should Base Traces remain at 🏅 to match Eth Traces?


59-59: Disregard static analysis warning on line 59.

The MD034 "bare URL used" flagged by markdownlint-cli2 is a false positive. URLs within markdown table cells are acceptable and do not violate MD034 guidelines.


Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands and usage tips.

@nikbhintade nikbhintade merged commit 8afaf2c into main Dec 17, 2025
3 checks passed
@nikbhintade nikbhintade deleted the nb/remove-alchemy-banner branch December 17, 2025 13:23
@DenhamPreen DenhamPreen mentioned this pull request Mar 9, 2026
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