Skip to content

docs: fix remaining broken and outdated links from #1170 - #1179

Closed
kutluhaneth46 wants to merge 1 commit into
base:mainfrom
kutluhaneth46:cursor/fix-outdated-docs-links-88c1
Closed

docs: fix remaining broken and outdated links from #1170#1179
kutluhaneth46 wants to merge 1 commit into
base:mainfrom
kutluhaneth46:cursor/fix-outdated-docs-links-88c1

Conversation

@kutluhaneth46

Copy link
Copy Markdown

Summary

#1173 already updates the reth pruning URLs in the env files. These three links from #1170 are still broken/outdated on main:

  1. README.mdhttps://basechaindata.vercel.app (404) → Base snapshots docs
  2. SECURITY.md — Optimism Bedrock docs URL (404) → current OP Stack getting-started page
  3. CONTRIBUTING.md — old Coinbase support portal URL (redirect) → https://help.coinbase.com/en

Note: #1174 removes the [3] definition in CONTRIBUTING.md while the body still references [our support team][3]. This PR keeps the reference and updates the destination instead.

Test plan

Fixes #1170

Update the three links that are still broken/outdated after base#1173:
- README: replace dead basechaindata.vercel.app with Base snapshots docs
- SECURITY.md: replace removed Optimism Bedrock docs URL
- CONTRIBUTING.md: replace redirected Coinbase support portal URL

Fixes base#1170
@cb-heimdall

Copy link
Copy Markdown
Collaborator

🟡 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

@danyalprout danyalprout closed this Sep 7, 2026
@kutluhaneth46

Copy link
Copy Markdown
Author

@danyalprout Noticed this was closed without merge — totally fine if it was intentional.

Quick status check against current main in case it helps: the three targets from #1170 still look broken/outdated on HEAD:

  • SECURITY.mdhttps://docs.optimism.io/docs/releases/bedrock/ (404)
  • CONTRIBUTING.md → old support.coinbase.com/... portal URL (dead)
  • README.md still points snapshot sizing at basechaindata.vercel.app (the docs.base.org snapshots page is the live one)

Happy to reopen/refresh this PR if useful, or stand down if you'd rather handle it in-tree. Thanks either way!

@danyalprout

Copy link
Copy Markdown
Collaborator

@kutluhaneth46 we're in the process of archiving this repo (to move all dev and releases to base/base). If they're still issues there, feel free to open a PR and @ me on it and we'll merge.

@kutluhaneth46

Copy link
Copy Markdown
Author

Thanks @danyalprout — checked base/base against the three #1170 leftovers:

  • SECURITY.md — no more docs.optimism.io/docs/releases/bedrock/ link
  • CONTRIBUTING.md — no more dead support.coinbase.com/.../portal/... URL
  • README.md — snapshots point at docs.base.org (not basechaindata.vercel.app)

Looks like those are already fixed in the new repo, so I won’t open a duplicate PR there. Appreciate the pointer on the archive move.

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.

docs: broken and outdated links in README, SECURITY.md, CONTRIBUTING.md, and env files

4 participants