Skip to content

Offerings CRUD on the Models page + enable/disable toggle - #1

Merged
jsaigou merged 1 commit into
mainfrom
offerings-crud-models-page
Sep 6, 2026
Merged

Offerings CRUD on the Models page + enable/disable toggle#1
jsaigou merged 1 commit into
mainfrom
offerings-crud-models-page

Conversation

@jsaigou

@jsaigou jsaigou commented Sep 6, 2026

Copy link
Copy Markdown
Owner

Summary

  • Models -> Offerings was read-only; admins now get full create/edit/delete for offerings directly on that page, sharing a new components/catalog/OfferingForm.tsx with Settings -> Catalog -> Offerings.
  • Adds a per-row Enable/Disable toggle (resends the full record, since the update endpoint is a full replace, not a merge).
  • No backend changes.

Test plan

  • tsc -b --force, vite build, oxlint clean
  • Live-verified against a running instance: create, edit, delete, and enable/disable toggle all confirmed working
  • Confirmed via API that toggling enabled does not blank any other field on the row

🤖 Generated with Claude Code

Models -> Offerings was previously read-only, so there was no way to add
an offering or repoint one at a different model/variant without going to
Settings -> Catalog -> Offerings. Extract OfferingForm into
components/catalog/OfferingForm.tsx (now shared by both surfaces) and give
admins full create/edit/delete plus visibility into disabled offerings on
the Models page. Also adds a quick per-row Enable/Disable toggle next to
Edit/Delete — since the update endpoint is a full replace rather than a
merge, the toggle resends the whole record with only `enabled` flipped.

No backend changes required.

Claude-Session: https://claude.ai/code/session_01Wo5BP532fo77c4NysdguC1
@jsaigou
jsaigou merged commit 92beaa3 into main Sep 6, 2026
2 checks passed
@jsaigou
jsaigou deleted the offerings-crud-models-page branch September 6, 2026 04:47
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.

1 participant