Skip to content

Conversation

@NullSoldier
Copy link
Contributor

What changed? Why?

This was referencing an old name GetEASAttestationOptions which has since been renamed and resulting in a broken link.

How has it been tested?

  1. npx mint dev
  2. Open http://localhost:3000/onchainkit/identity/get-attestations
  3. Click GetAttestationOptions (at bottom of page, and in returns for GetAttestations())
  4. See link is broken before, and fixed after

@cb-heimdall
Copy link
Collaborator

cb-heimdall commented Oct 7, 2025

✅ Heimdall Review Status

Requirement Status More Info
Reviews 1/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

This was referencing an old name GetEASAttestationOptions which has
since been renamed and resulting in a broken link.
@NullSoldier NullSoldier force-pushed the jason/fix-broken-links branch from 1488470 to 7506347 Compare October 18, 2025 03:33
@NullSoldier NullSoldier requested a review from dgca October 18, 2025 03:33
@youssefea youssefea merged commit b14b549 into base:master Oct 20, 2025
3 checks passed
@NullSoldier NullSoldier deleted the jason/fix-broken-links branch October 24, 2025 20:18
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