dries/ocman
1
0
Fork 1
Code Issues 37 Pull requests 2 Projects Releases 122 Packages Wiki Activity Actions

refactor(ui): always group sessions tab by project #177

Merged
dries merged 1 commit from feat/sessions-always-grouped into main 2026-06-29 23:14:02 +02:00
Conversation 1 Commits 1 Files changed 2 +8 -29
dries commented 2026-06-29 16:33:28 +02:00
Owner
Copy link

Summary

  • Remove the flat (ungrouped) session list from the Sessions tab; it now always renders the project-grouped view.
  • Drop the "Group by project" toggle button.
  • Remove the now-unused dashboardGrouped / toggleDashboardGrouped state from uiStore (including its persist entry).

SessionTable is kept — still used by ProjectDetail and for shared exports.

Testing

  • make lint
  • make test (1320 frontend + Go suites pass)
## Summary - Remove the flat (ungrouped) session list from the Sessions tab; it now always renders the project-grouped view. - Drop the "Group by project" toggle button. - Remove the now-unused `dashboardGrouped` / `toggleDashboardGrouped` state from `uiStore` (including its persist entry). `SessionTable` is kept — still used by ProjectDetail and for shared exports. ## Testing - `make lint` - `make test` (1320 frontend + Go suites pass)
refactor(ui): always group sessions tab by project
Some checks failed
CI / Playwright E2E (pull_request) Failing after 2m10s
Details
CI / Frontend (pull_request) Failing after 2m13s
Details
CI / Backend (pull_request) Successful in 4m57s
Details
CI / Coverage Ratchet (pull_request) Has been skipped
Details
CI / Build (pull_request) Has been skipped
Details
CI / Build Desktop (macOS arm64) (pull_request) Has been skipped
Details
CI / Semantic Tag (pull_request) Has been skipped
Details
c5501f5bc4
dries force-pushed feat/sessions-always-grouped from c5501f5bc4
Some checks failed
CI / Playwright E2E (pull_request) Failing after 2m10s
Details
CI / Frontend (pull_request) Failing after 2m13s
Details
CI / Backend (pull_request) Successful in 4m57s
Details
CI / Coverage Ratchet (pull_request) Has been skipped
Details
CI / Build (pull_request) Has been skipped
Details
CI / Build Desktop (macOS arm64) (pull_request) Has been skipped
Details
CI / Semantic Tag (pull_request) Has been skipped
Details
to d4fef412ea
All checks were successful
CI / Backend (pull_request) Successful in 2m34s
Details
CI / Frontend (pull_request) Successful in 3m10s
Details
CI / Build Desktop (macOS arm64) (pull_request) Successful in 49s
Details
CI / Build (pull_request) Successful in 3m42s
Details
CI / Playwright E2E (pull_request) Successful in 6m55s
Details
CI / Semantic Tag (pull_request) Has been skipped
Details
CI / Coverage Ratchet (pull_request) Successful in 5m5s
Details
2026-06-29 16:46:40 +02:00 Compare
forgejo-actions commented 2026-06-29 16:54:55 +02:00
Copy link

Coverage ratchet: ✅ pass

Suite Baseline This PR Δ
go 69.40% 69.40% +0.00 ✅
frontend 57.39% 57.41% +0.02 ✅

Tolerance: -0.1%. Baseline stored on gh-pages.

<!-- coverage-ratchet --> ### Coverage ratchet: ✅ pass | Suite | Baseline | This PR | Δ | | |---|---|---|---|---| | go | 69.40% | 69.40% | +0.00 | ✅ | | frontend | 57.39% | 57.41% | +0.02 | ✅ | _Tolerance: -0.1%. Baseline stored on `gh-pages`._
dries merged commit d2081b0fcf into main 2026-06-29 23:14:02 +02:00
Sign in to join this conversation.
Reviewers
No reviewers
Labels
Clear labels   
backend

Go backend code

  
bug

Something is broken or incorrect

  
chore

Maintenance, tooling, or cleanup

  
duplication

Code duplication to be addressed

  
effort:complex

Significant effort or risk

  
effort:medium

Moderate effort required

  
effort:trivial

Quick fix, low complexity

  
enhancement

New feature or capability

  
frontend

TypeScript / React frontend code

  
fullstack

Touches both backend and frontend

  
priority:high

Should be addressed soon

  
ready-for-agent

Agent-grabbable ticket, fully specced

  
refactor

Code restructuring without behaviour change

  
security

Security related

No labels
backend
bug
chore
duplication
effort:complex
effort:medium
effort:trivial
enhancement
frontend
fullstack
priority:high
ready-for-agent
refactor
security
Milestone
Clear milestone
No items
No milestone
Projects
Clear projects
No items
No project
Assignees
Clear assignees
No assignees
2 participants
Notifications
Due date
The due date is invalid or out of range. Please use the format "yyyy-mm-dd".

No due date set.

Dependencies

No dependencies set.

Reference
dries/ocman!177
No description provided.