Skip to content

Conversation

@wilsoncusack
Copy link
Contributor

What changed? Why?
In this PR I

  • Change chainID to Base chain ID
  • Update auth diagram and code sample to use local nonce generation
  • Added note about popup blocker issues
  • Removed deadlink

Notes to reviewers
Locally

How has it been tested?

@cb-heimdall
Copy link
Collaborator

cb-heimdall commented Jul 22, 2025

🟡 Heimdall Review Status

Requirement Status More Info
Reviews 🟡 0/1
Denominator calculation
Show calculation
1 if user is bot 0
1 if user is external 0
2 if repo is sensitive 0
From .codeflow.yml 1
Additional review requirements
Show calculation
Max 0
0
From CODEOWNERS 0
Global minimum 0
Max 1
1
1 if commit is unverified 0
Sum 1

@cb-heimdall
Copy link
Collaborator

Review Error for youssefea @ 2025-07-22 17:39:23 UTC
User cannot review their own commit

@youssefea youssefea merged commit 4f28cff into master Jul 22, 2025
6 of 7 checks passed
@youssefea youssefea deleted the wilson/chain-id branch July 22, 2025 17:41
Zdonuk added a commit to Zdonuk/docs that referenced this pull request Oct 28, 2025
Add specialized guide for Base node sync issues based on
community reports (base#127, base#251, base#369, base#413, base#419, base#433).

Includes:
- 4 sync scenarios with Check/Action solutions
- Hardware anti-patterns (RAID-5, NAS, SATA warnings)
- Reth performance optimization
- Monitoring commands and quick reference
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.

4 participants