feat: add edit support for env vars, labels, and volumes (closes #67) #76

clawbot · 2026-02-16T09:25:49+01:00

clawbot commented

2026-02-16 09:25:49 +01:00

Summary

Adds inline edit functionality for environment variables, labels, and volume mounts.

Changes

Routes: Added POST /apps/{id}/env-vars/{varID}/edit, /labels/{labelID}/edit, /volumes/{volumeID}/edit
Handlers: HandleEnvVarEdit(), HandleLabelEdit(), HandleVolumeEdit() — find entity, validate ownership, update via Save()
Templates: Alpine.js inline edit toggle — click Edit to show form with current values, Save/Cancel buttons

Notes

Models already had Save() with update support (when ID != 0), so no model changes needed
Pre-existing gofmt issues in 3 other files (middleware.go, app.go, deploy.go) — not touched by this PR

Test Results

all tests pass — make test ✅
gofmt clean on changed files ✅
pre-existing format issues in unrelated files (not introduced by this PR)

Closes #67

## Summary Adds inline edit functionality for environment variables, labels, and volume mounts. ## Changes - **Routes**: Added `POST /apps/{id}/env-vars/{varID}/edit`, `/labels/{labelID}/edit`, `/volumes/{volumeID}/edit` - **Handlers**: `HandleEnvVarEdit()`, `HandleLabelEdit()`, `HandleVolumeEdit()` — find entity, validate ownership, update via `Save()` - **Templates**: Alpine.js inline edit toggle — click Edit to show form with current values, Save/Cancel buttons ## Notes - Models already had `Save()` with update support (when ID != 0), so no model changes needed - Pre-existing `gofmt` issues in 3 other files (middleware.go, app.go, deploy.go) — not touched by this PR ## Test Results ``` all tests pass — make test ✅ gofmt clean on changed files ✅ pre-existing format issues in unrelated files (not introduced by this PR) ``` Closes #67

sneak was assigned by clawbot

2026-02-16 09:25:49 +01:00

clawbot added 1 commit 2026-02-16 09:25:50 +01:00

feat: add edit support for env vars, labels, and volumes 8156305705

- Add POST /apps/{id}/env-vars/{varID}/edit endpoint
- Add POST /apps/{id}/labels/{labelID}/edit endpoint
- Add POST /apps/{id}/volumes/{volumeID}/edit endpoint
- Add inline edit UI with Alpine.js toggle in app_detail template
- Models already support Save() with update when ID != 0

Closes #67

sneak commented

2026-02-16 09:52:00 +01:00

fix merge conflicts pls

clawbot commented

2026-02-16 09:57:34 +01:00

After rebasing onto main, this PR is empty — all the edit handler code (env vars, labels, volumes) is already on main. The feature was merged as part of the earlier PRs (#67 implementation). The merge conflicts were just from the branch being stale.

This PR can be closed.

After rebasing onto main, this PR is empty — all the edit handler code (env vars, labels, volumes) is already on main. The feature was merged as part of the earlier PRs (#67 implementation). The merge conflicts were just from the branch being stale. This PR can be closed.

clawbot closed this pull request

2026-02-19 22:40:00 +01:00

Pull request closed

Please reopen this pull request to perform a merge.

Sign in to join this conversation.

2 Participants

Notifications

Due Date

No due date set.

Dependencies

No dependencies set.

Reference: sneak/upaas#76