Skip to content

fix(nav): move /transactions from Insights to Activity#656

Merged
Oolab-labs merged 1 commit into
mainfrom
feat/nav-transactions-to-activity
May 19, 2026
Merged

fix(nav): move /transactions from Insights to Activity#656
Oolab-labs merged 1 commit into
mainfrom
feat/nav-transactions-to-activity

Conversation

@kungfuk3nnyyy
Copy link
Copy Markdown
Collaborator

Summary

`/transactions` is a per-write audit log (one row per disk-touching action) — same shape as Traces, not an aggregate. Audit caught it nested under Insights next to Analytics + Approval Insights, where users browsing aggregates didn't expect a raw event log.

Move to Activity, next to Traces. Single source of truth: sidebar + command palette + mobile bottom-nav all pick up the change.

Test plan

  • Sidebar: /transactions appears under Activity, not Insights
  • ⌘K: "Activity — Transactions" entry shows up
  • /transactions still loads (route unchanged, just nav label moved)

🤖 Generated with Claude Code

/transactions is a per-write audit log (one row per disk-touching
action) — same shape as Traces, not an aggregate metric. Audit caught
it nested under Insights where Analytics + Approval Insights actually
live; users browsing aggregates didn't expect a raw event log there.

Move to Activity next to Traces. Single source of truth means
sidebar + command palette + mobile bottom-nav all pick up the change.

Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
@Oolab-labs Oolab-labs merged commit 9f13249 into main May 19, 2026
18 checks passed
@Oolab-labs Oolab-labs deleted the feat/nav-transactions-to-activity branch May 19, 2026 10:25
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.

2 participants