Skip to content

TUI: add nested store modal to the profile form#234

Merged
rmanibus merged 1 commit into
mainfrom
feat/tui-profile-store-modal
Apr 4, 2026
Merged

TUI: add nested store modal to the profile form#234
rmanibus merged 1 commit into
mainfrom
feat/tui-profile-store-modal

Conversation

@rmanibus
Copy link
Copy Markdown
Contributor

@rmanibus rmanibus commented Apr 4, 2026

Summary

  • let the profile form create a store inline instead of requiring an existing store
  • add a nested store modal and return to the profile form with the new store selected
  • allow editing the currently selected store from the profile modal

Scope

  • type-driven store modal for local, s3, b2, and sftp URIs
  • profile modal store selector now includes + Create store
  • preserve existing extra store settings on edit while updating the URI

Testing

  • env GOCACHE=/tmp/cloudstic-gocache go test -count=1 ./cmd/cloudstic ./internal/app ./internal/tui
  • env GOCACHE=/tmp/cloudstic-gocache GOLANGCI_LINT_CACHE=/tmp/cloudstic-golangci-lint golangci-lint run ./cmd/cloudstic ./internal/app ./internal/tui

Related: #232

@rmanibus rmanibus added the enhancement New feature or request label Apr 4, 2026
@rmanibus rmanibus linked an issue Apr 4, 2026 that may be closed by this pull request
@rmanibus rmanibus added this to the v1.15.0 milestone Apr 4, 2026
@codecov
Copy link
Copy Markdown

codecov Bot commented Apr 4, 2026

Codecov Report

❌ Patch coverage is 47.24638% with 182 lines in your changes missing coverage. Please review.

Files with missing lines Patch % Lines
cmd/cloudstic/cmd_tui_store_form.go 45.27% 119 Missing and 20 partials ⚠️
cmd/cloudstic/cmd_tui_profile_form.go 53.75% 25 Missing and 12 partials ⚠️
internal/app/tui_service.go 45.45% 3 Missing and 3 partials ⚠️

📢 Thoughts on this report? Let us know!

@rmanibus rmanibus merged commit 08a70e9 into main Apr 4, 2026
10 of 11 checks passed
@rmanibus rmanibus deleted the feat/tui-profile-store-modal branch April 4, 2026 11:22
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

enhancement New feature or request

Projects

None yet

Development

Successfully merging this pull request may close these issues.

TUI: add nested store create/edit flow from profile form

1 participant