Skip to content

Fix "onchain score" component by updating to Etherscan API V2#2684

Merged
amiecorso merged 5 commits intomasterfrom
amie/fix-onchain-score-heatmap
Oct 3, 2025
Merged

Fix "onchain score" component by updating to Etherscan API V2#2684
amiecorso merged 5 commits intomasterfrom
amie/fix-onchain-score-heatmap

Conversation

@amiecorso
Copy link
Contributor

@amiecorso amiecorso commented Oct 2, 2025

What changed? Why?
This PR updates the route that feeds our onchain score component at base.org/name/YOUR_BASENAME.

Notes to reviewers

How has it been tested?
Before:
Screenshot 2025-10-02 at 3 37 52 PM

After:
Screenshot 2025-10-02 at 3 37 39 PM

Have you tested the following pages?

BaseWeb

  • [] base.org
  • [] base.org/names
  • [] base.org/builders
  • [] base.org/ecosystem
  • base.org/name/jesse
  • [] base.org/manage-names
  • [] base.org/resources

@cb-heimdall
Copy link
Collaborator

cb-heimdall commented Oct 2, 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

@vercel
Copy link

vercel bot commented Oct 2, 2025

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Preview Comments Updated (UTC)
base-web Ready Ready Preview Comment Oct 3, 2025 10:21pm

@amiecorso amiecorso marked this pull request as ready for review October 2, 2025 23:23
import { isAddress } from 'viem';

const ETHERSCAN_API_KEY = process.env.ETHERSCAN_API_KEY;
const BASESCAN_API_KEY = process.env.BASESCAN_API_KEY;
Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

all uses ETHERSCAN API key now

Copy link
Collaborator

@arjun-dureja arjun-dureja left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@amiecorso amiecorso merged commit 26529ba into master Oct 3, 2025
11 checks passed
@amiecorso amiecorso deleted the amie/fix-onchain-score-heatmap branch October 3, 2025 23:00
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