Skip to content

feat(nav): add AMP and Geothermal sidebar groups #363

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Open
jirhiker wants to merge 6 commits into
base: staging
Choose a base branch
from
Open

feat(nav): add AMP and Geothermal sidebar groups #363

Show file tree
Hide file tree
Changes from all commits
Commits
Show all changes
6 commits
Select commit Hold shift + click to select a range
60568a9
feat(nav): add the Geothermal nav group
jirhiker Aug 27, 2026
507d7e8
feat(nav): group Wells, Field Sheets and Contacts under AMP
jirhiker Aug 31, 2026
bf3494c
fix(nav): let sidebar groups collapse, and stop them fighting each other
jirhiker Aug 31, 2026
a033a34
fix(nav): group headers toggle instead of navigating
jirhiker Aug 31, 2026
da690cf
style(nav): divide the sidebar groups from the flat resources
jirhiker Aug 31, 2026
253e265
feat(nav): show the Geothermal group everywhere
jirhiker Aug 31, 2026
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
4 changes: 2 additions & 2 deletions AGENTS.md
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -120,8 +120,8 @@ UI layering is documented in `FRONTEND.md` — read it before adding styles. Sho
In code, that means WIP surfaces gate on dev or preview, never on staging:

```ts
export const SHOW_WIP_FEATURES =
import.meta.env.DEV || import.meta.env.VITE_APP_ENV === 'preview'
export const BYPASS_GEOTHERMAL_GATE =
import.meta.env.DEV === true || import.meta.env.VITE_APP_ENV === 'preview'
```

`recordsGridLogic.ts` gates exactly this way. Do not add `'staging'` to that check, and do not add a staging arm to a new one.
Expand Down
8 changes: 4 additions & 4 deletions docs/geothermal-well-inventory.md
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -145,8 +145,8 @@ as the records grid (bypassed in local dev, enforced in prod).
## 8. Navigation & route

- Route: `/geothermal/wells/inventory`.
- Nav: none yet. The page is reachable by URL while the work is in progress;
it gets a nav entry when a geothermal nav group lands.
- Nav: an **Inventory** entry under the **Geothermal** nav group, alongside
Records and Temp-Depth.

---

Expand All @@ -163,7 +163,7 @@ as the records grid (bypassed in local dev, enforced in prod).
- Boolean, date, and **dropdown** cell kinds in `EditableDataGrid`, plus
allowed-value lists for the enum fields.
- Inventory page (grid + toolbar: Add rows, Upload CSV, Download template, Save).
- Route (no nav entry yet).
- Route + nav entry (Geothermal group).
- Create-only save wrapper (adapt records-grid save to POST-only).

---
Expand All @@ -175,7 +175,7 @@ as the records grid (bypassed in local dev, enforced in prod).
surface the conflict inline. No client-side dedupe/upsert in v1. ✔
- **Enum fields** (`well_type`, `well_class`, `status`): **dropdowns** from
fixed allowed-value lists (new select cell kind). ✔
- **Nav placement:** none yet — URL-only until a geothermal nav group lands. ✔
- **Nav placement:** Geothermal nav group. ✔
- **Records vs inventory:** inventory is create-wells only; the existing records
grid is untouched. ✔

Expand Down
Loading
Loading