Polish Claim Beta copy for ERC-8004 tracking#392
Conversation
|
The latest updates on your projects. Learn more about Vercel for GitHub.
|
There was a problem hiding this comment.
💡 Codex Review
Here are some automated review suggestions for this pull request.
Reviewed commit: 860f5d3c7d
ℹ️ About Codex in GitHub
Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you
- Open a pull request for review
- Mark a draft as ready
- Comment "@codex review".
If Codex has suggestions, it will comment; otherwise it will react with 👍.
Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".
| <div class="activation-cta"> | ||
| <h3>Activation pipeline</h3> | ||
| <p id="pipelineIntro">Submit a claim to load persisted payment, ENS, IPFS, genesis, and tenant proof status.</p> | ||
| <p id="pipelineIntro">Submit or import a claim to view activation status across ENS records, agent card pinning, ERC-8004 tracking, genesis receipt, tenant proof, and first-action receipt.</p> |
There was a problem hiding this comment.
Render ERC-8004 status before promising it in the pipeline
After a user submits or imports a claim with pinned cards, /api/claims/status returns the registration records in data.registrations, but renderActivationPipeline() only renders the existing nine pipeline fields and never reads those records. Consequently, the public Claim flow cannot display the advertised ERC-8004 registration, agent ID, or ENSIP-25 status—even when that status is available—so this text promises a tracking view that users do not receive.
Useful? React with 👍 / 👎.
Motivation
prepared/pinned/tracking/registered/verified).Description
Claim Beta · ENS + ERC-8004 discovery tracking · CLAS-ready agentsand to a customer-facing subheadline describing activation, signer records, pinning, and ERC-8004 metadata preparation..discovery-panel+.discovery-status) that explains current status items (cards can be generated/pinned, tracking rows prepared, onchain registration pending, ENSIP-25 unverified).acmeexamples and soften managed/assisted activation copy to state operator-reviewed provisioning and that onchain writes/registration/verifications are marked complete only after confirmation.Claim Betaframing and remove messaging that implied immediate checkout availability; update the activation checklist and agent-card/ENS instruction copy accordingly.activationModeInstructiontext and the conditional string used bybuildENSRecords) and add responsive CSS for the new panel, while leaving IDs, classes, function names, API paths, and wizard flow unchanged.Testing
npm test: 165 of 169 tests passed and 4 tests failed; failures are related to ERC-8004/pinned-card orchestration and registry/pinning configuration (these failures are unrelated to thepublic/claim.htmlcopy changes).npm run check:linkswhich passed and confirmed local links/assets resolve.walletConnectBtn,keygenResult,ensRecords,sdkConfig,agentCardJson,summaryGrid, andactivationPipeline).node --check /tmp/claim-inline.jsto ensure inline script syntax is valid and rangit diff --checkto ensure no whitespace/errors introduced; both checks passed; a browser screenshot check was not possible due to no headless browser binary in the environment.Codex Task