Skip to content

Remove dead NOTIFICATIONS.md links in minikit-to-farcaster mapping#91

Open
jayadiki wants to merge 1 commit into
base:masterfrom
jayadiki:patch-1
Open

Remove dead NOTIFICATIONS.md links in minikit-to-farcaster mapping#91
jayadiki wants to merge 1 commit into
base:masterfrom
jayadiki:patch-1

Conversation

@jayadiki

@jayadiki jayadiki commented Jun 10, 2026

Copy link
Copy Markdown

The useNotification section in the minikit-to-farcaster mapping links to NOTIFICATIONS.md twice but that file does not exist in the repo and is not listed in the migration overview. Both links are dead. Removed the two references. The section still explains notifications go through a server webhook and keeps the before and after code examples.

Type of change: documentation. Affected skill: build-on-base.

The useNotification section linked to NOTIFICATIONS.md twice but that file does not exist in the repo and is not listed in the migration overview. Removed the two dead references. The section still explains notifications go through a server webhook and keeps the before and after examples.
@cb-heimdall

cb-heimdall commented Jun 10, 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

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