Skip to content

docs: fix broken links in minikit-to-farcaster migration guide - #140

Closed
memosr wants to merge 1 commit into
base:masterfrom
memosr:fix/broken-doc-links
Closed

memosr wants to merge 1 commit into
base:masterfrom
memosr:fix/broken-doc-links

Conversation

@memosr

@memosr memosr commented Jul 8, 2026

Copy link
Copy Markdown

Fixes three broken markdown links in the minikit-to-farcaster migration references. All target files verified on disk (case-sensitive) and the external URL fetched live.

  • dependencies.md: AUTH.md to auth.md (actual file is lowercase; 404 on GitHub / case-sensitive filesystems)
  • examples.md: PROVIDER.md to provider.md (same case-sensitivity issue)
  • mapping.md (x2): NOTIFICATIONS.md does not exist anywhere in the repo; repointed to the official Farcaster notifications guide (verified live), consistent with the existing miniapps.farcaster.xyz link in dependencies.md

Docs-only, no code changes.

@cb-heimdall

cb-heimdall commented Jul 8, 2026 •

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

- dependencies.md: AUTH.md -> auth.md (case-sensitive fix)
- examples.md: PROVIDER.md -> provider.md (case-sensitive fix)
- mapping.md: NOTIFICATIONS.md (nonexistent) -> official Farcaster notifications guide
@memosr
memosr force-pushed the fix/broken-doc-links branch from c2669af to 7e609dc Compare August 16, 2026 21:41
@memosr

memosr commented Oct 24, 2026

Copy link
Copy Markdown
Author

Closing: the migration guide this fixed was removed in #162.

@memosr memosr closed this Oct 24, 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