[#538] Update PL_TEST → PLOT token references#541
Conversation
- Update PLOT_TOKEN mainnet address to 0x4F567DACBF9D15A6acBe4A47FC2Ade0719Fb63C4 - Simplify RESERVE_LABEL to always "PLOT" (no testnet conditional) - lib/usd-price.ts automatically picks up new address via PLOT_TOKEN import - Bump version to 0.1.10 Fixes #538 Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
|
The latest updates on your projects. Learn more about Vercel for GitHub. |
project7-interns
left a comment
There was a problem hiding this comment.
APPROVE — minimal config update. PLOT token address matches new deployment (0x4F567DAC...), RESERVE_LABEL simplified to always 'PLOT' per mainnet-only direction. No issues.
project7-interns
left a comment
There was a problem hiding this comment.
Verdict: REQUEST CHANGES
Summary
The mainnet PLOT_TOKEN constant update is correct, and the token page / swap UI already derive from that constant or already point at the right PLOT URLs. But issue #538 explicitly requires removing all remaining old-PL_TEST references, and one still remains in the repo.
Findings
- [medium]
.env.examplestill documents the old PL_TEST mainnet address.- File:
.env.example:50 - Suggestion: Update the Base Mainnet comment to the real PLOT token
0x4F567DACBF9D15A6acBe4A47FC2Ade0719Fb63C4and remove the stalePL_TESTlabel so the repo no longer contains the old address.
- File:
Decision
Requesting changes because issue #538 and its acceptance criteria explicitly require replacing all old PL_TEST address references, and .env.example still contains one.
Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
project7-interns
left a comment
There was a problem hiding this comment.
Verdict: APPROVE
Summary
The updated head resolves the last remaining old-PL_TEST reference. The repo now consistently points at the real PLOT mainnet token across constants and documented env values.
Findings
- No blocking findings.
Decision
Approve. .env.example now documents # PLOT_TOKEN: 0x4F567DACBF9D15A6acBe4A47FC2Ade0719Fb63C4, which clears the only remaining stale reference I found in issue #538.
Summary
PLOT_TOKENmainnet address from0xF8A2C39...to0x4F567DAC...RESERVE_LABELto always"PLOT"(removed testnet conditional)lib/usd-price.tsautomatically picks up new address viaPLOT_TOKENimport0.1.10Test plan
Fixes #538
🤖 Generated with Claude Code