Skip to content

docs(builder-codes): drop deprecated base.dev attribution check from sub-pages - #2066

Open
youssefea wants to merge 1 commit into
base:masterfrom
youssefea:fix/builder-codes-remove-deprecated-basedev-verification
Open

youssefea wants to merge 1 commit into
base:masterfrom
youssefea:fix/builder-codes-remove-deprecated-basedev-verification

Conversation

@youssefea

Copy link
Copy Markdown
Contributor

What changed? Why?

Removes the deprecated "Check base.dev" step from the Verify attribution sections of the two Builder Codes sub-pages, and renumbers the remaining steps (block explorer, then open-source tools):

  • docs/specifications/builder-codes/for-app-developers.mdx
  • docs/specifications/builder-codes/for-agent-developers.mdx

#2024 (merged 2026-10-01, by a Builder Codes maintainer) removed this exact step from builder-codes/overview.mdx and gave the reason "This is now deprecated". The same step was still on these two pages, which sent developers to the deprecated base.dev view to check attribution. After this change, all three pages give the same steps.

Supporting sources:

Notes to reviewers

How has it been tested?

  • node scripts/lint-mdx.js on both changed files: 0 errors, 0 warnings
  • node scripts/check-terminology.js on both changed files: passed
  • npm test: 113 passed, 0 failed
  • git grep "Check base.dev": no matches remain

Screenshots

Not captured: the local Mintlify preview wasn't run for this automated change. The rendered change only removes one step from each list and renumbers the rest, as shown in the diff.

Generated with Toshi

…sub-pages

Co-authored-by: Toshi <toshi-noreply@coinbase.com>
@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

youssefea added a commit that referenced this pull request Oct 6, 2026
Base Dashboard 2.0 no longer has the Onchain transaction-type view, so the
Check base.dev step sent developers to a screen that no longer exists.
Matches the overview page (#2024) and supersedes #2066.

Co-authored-by: Toshi <toshi-noreply@coinbase.com>
youssefea added a commit that referenced this pull request Oct 6, 2026
* docs(builder-codes): add user analytics guide for app developers

Adds a Track User Analytics section to the Builder Codes for App Developers page with a copy-paste prompt builders can give their coding agent to set up product analytics.

* docs(builder-codes): make user analytics section tool-agnostic

Addresses review feedback: removes the agent prompt, analytics tool names and Wagmi references; replaces them with steps describing what to measure and which onchain signals to use.

* docs(builder-codes): drop deprecated base.dev attribution check

Base Dashboard 2.0 no longer has the Onchain transaction-type view, so the
Check base.dev step sent developers to a screen that no longer exists.
Matches the overview page (#2024) and supersedes #2066.

Co-authored-by: Toshi <toshi-noreply@coinbase.com>

* docs(builder-codes): show where each analytics signal comes from

Replaces the generic Track User Analytics steps with the concrete data
sources behind each journey stage:

- read the user identifier from the EIP-1193 provider (eth_requestAccounts,
  accountsChanged, chainChanged) and explain smart account addresses
- list the in-app signal for open, connect, submit and success, including
  the wallet_sendCalls bundle ID and wallet_getCallsStatus resolution and
  EIP-1193 error code 4001
- explain where the user, Builder Code and success live for EOA
  transactions vs ERC-4337 UserOperations, the ERC-8021 suffix byte
  layout, and a Viem + ox function that returns one row per user action
- define the active user, retention and funnel calculations

Drops the guidance on which identifiers to collect; the section is about
finding the right data, not prescribing a data policy.

Co-authored-by: Toshi <toshi-noreply@coinbase.com>

* docs(builder-codes): update Dashboard references for Dashboard 2.0

Base Dashboard 2.0 replaced the transaction and user counts with spot
trading volume, borrow TVL and lending TVL derived from Builder Code
activity, renamed apps to projects, and moved the code to
Settings > Project Settings > Builder Code.

- point the Builder Code location and code comments to the new path
- document the Encoded String format (the ready-made dataSuffix)
- replace the analytics benefit claims on the overview, app and agent
  pages with what the Dashboard shows today, linking to Track User
  Analytics for users, retention and conversion

Co-authored-by: Toshi <toshi-noreply@coinbase.com>

* docs: link to dashboard.base.org instead of base.dev

base.dev now 302-redirects to dashboard.base.org. Points the Builder Codes
links and the two data-driven-growth redirects straight at the canonical
host. blog.base.dev, verify.base.dev and api.base.dev are separate
services and are unchanged.

Co-authored-by: Toshi <toshi-noreply@coinbase.com>

* docs(builder-codes): drop Base Account example from smart account note

Co-authored-by: Toshi <toshi-noreply@coinbase.com>

---------

Co-authored-by: Youssef El Allali <youcefea99@gmail.com>
Co-authored-by: Toshi <toshi-noreply@coinbase.com>

This branch has not been deployed

No deployments
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