fix: UI fixes — remove API link, fix About datasets, add contribution CTA#38
fix: UI fixes — remove API link, fix About datasets, add contribution CTA#38victorquinn merged 1 commit intomainfrom
Conversation
1. Remove non-functional API link from top nav and hero CTA 2. Add missing datasets to About page (Utilities, Programs, Rates, Territories) 3. Merge browse buttons into dataset cards (clickable cards with Browse link) 4. Add community contribution section to About page (OSM-style call to action) 5. Fix misleading 'Updated Daily' to 'Continuously Updated' 6. Replace API docs link in footer with Contribute link
|
The latest updates on your projects. Learn more about Vercel for GitHub.
|
![chatgpt-codex-connector[bot]](https://avatars.githubusercontent.com/in/1144995?s=60&v=4){kind=small}
There was a problem hiding this comment.
💡 Codex Review
Here are some automated review suggestions for this pull request.
Reviewed commit: 58c35729d5
ℹ️ 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".
| { label: "EV Charging Stations", value: "85,425", icon: "🔋", href: "/ev-charging" }, | ||
| { label: "Pricing Nodes", value: "4,065", icon: "💰", href: "/pricing-nodes" }, | ||
| { label: "Programs & Incentives", value: "500+", icon: "📋", href: "/explore?view=programs" }, | ||
| { label: "Rates & Tariffs", value: "~12k", icon: "📄", href: "/explore?view=rates" }, |
There was a problem hiding this comment.
This new CTA points to /explore?view=rates, but parseTab in components/explorer/ExplorerContext.tsx only recognizes utilities, grid-operators, power-plants, programs, and transmission-lines (and otherwise falls back to utilities). As a result, clicking "Rates & Tariffs" lands users on the utilities view instead of rates content, so this link currently misroutes the user flow.
Useful? React with 👍 / 👎.
2 participants
Changes
1. Remove non-functional API link ✅
docs.commongrid.devdoesn't exist)2-3. Fix About page dataset coverage ✅
Was only showing 6 highlights (Grid Operators, Power Plants, Transmission Lines, EV Charging, Pricing Nodes, Territory Boundaries).
Now shows all 9: added Utilities, Programs & Incentives, Rates & Tariffs
4-5. Merge browse buttons into cards ✅
Removed the redundant row of Badge buttons below the dataset cards. Each card is now a clickable link with a 'Browse X →' CTA built in.
6. Changelog — it's real ✅
The
generate-changelog.tsscript genuinely diffs data snapshots. But it only runs manually after syncs, not on a cron. Changed 'Updated Daily' to 'Continuously Updated' since data sources update on different cadences (EIA annually, AFDC weekly, etc).7. Community contribution CTA ✅
Added an OpenStreetMap-style section to the About page: 'Help Build CommonGrid' with calls to action for data corrections, new sources, tool improvements, and issue reporting.
8. PostHog — needs API key
Victor: I'll need your PostHog project API key to add the analytics snippet. Drop it when you have it and I'll add in a follow-up commit.
Build
tsc --noEmit✅ |next build✅