Skip to content

Conversation

@0xlukem
Copy link
Contributor

@0xlukem 0xlukem commented Dec 3, 2025

πŸ“ Description

This PR fixes some broken URLs flagged by Lychee and closes #1299
Also closes #1303

πŸ” Review Preference

Choose one:

  • βœ… I have time to handle formatting/style feedback myself
  • ⚑ Docs team handles formatting (check "Allow edits from maintainers")

πŸ€– AI-Ready Docs

If content changed, regenerate AI files:

  • βœ… I ran python3 scripts/generate_llms.py
  • ⚑ Docs team will regenerate (check "Allow edits from maintainers")

βœ… Checklist

@0xlukem 0xlukem requested a review from a team as a code owner December 3, 2025 16:46
Copilot AI review requested due to automatic review settings December 3, 2025 16:46
@0xlukem 0xlukem added B0 - Needs Review Pull request is ready for review C1 - Medium Medium priority task A3 - Quick Fixes Pull request contains small fixes (i.e., fixing a typo, removing old images, etc.) labels Dec 3, 2025
Copilot finished reviewing on behalf of 0xlukem December 3, 2025 16:48
Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

This PR addresses broken URLs identified by Lychee link checker, specifically removing an inaccessible Stakeworld snapshot provider reference and updating Wagmi documentation.

Key Changes:

  • Removed broken Stakeworld snapshot service URL from validator documentation
  • Updated Wagmi documentation to reference useConnection hook with v3 migration note
  • Regenerated AI-ready documentation files to reflect source changes

Reviewed changes

Copilot reviewed 9 out of 9 changed files in this pull request and generated 1 comment.

Show a summary per file
File Description
infrastructure/running-a-validator/onboarding-and-offboarding/start-validating.md Removed inaccessible Stakeworld URL from snapshot provider list
develop/smart-contracts/libraries/wagmi.md Updated documentation to reference useConnection hook with v3 migration note
llms-full.jsonl Auto-generated: Updated to reflect source file changes
.ai/pages/*.md Auto-generated: Updated to reflect source file changes
.ai/categories/*.md Auto-generated: Updated to reflect source file changes
.ai/site-index.json Auto-generated: Updated metadata and hashes

πŸ’‘ Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

@0xlukem 0xlukem requested a review from eshaben December 3, 2025 16:49
Copy link
Collaborator

@eshaben eshaben left a comment

Choose a reason for hiding this comment

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

@0xlukem because the examples do not currently use v3 (separate PR open against staging branch for this rn), let's only update this to be the v2 link and remain as useAccount until we merge staging. The staging branch has the v2 link in it right now, so you can grab it from there. Thanks!

@0xlukem 0xlukem requested a review from eshaben December 4, 2025 12:52
@eshaben eshaben requested a review from a team December 4, 2025 15:25
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

A3 - Quick Fixes Pull request contains small fixes (i.e., fixing a typo, removing old images, etc.) B0 - Needs Review Pull request is ready for review C1 - Medium Medium priority task

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Broken Links Detected on 2025-12-04 Broken Links Detected on 2025-12-03

3 participants