fix(approvals): Approval Center triage + drawer readability pass (#2762 P1-2/3/4/5, P2)#2803
Merged
Merged
Conversation
… P1-2/3/4/5, P2) - Inbox/cards surface the request amount inline (from the snapshot, preferring server-formatted payload_display); sort adds oldest-first and amount high→low next to newest-first (P1-3) - Flow-/system-initiated requests read "Flow-initiated" with a workflow icon instead of a bare person icon + "—" (desktop, mobile, drawer) (P1-4) - "Waiting on" chips dedupe a repeated approver to one chip with a ×N count, tooltip keeping every underlying id (P1-2) - DeclaredActionsBar maps the spec action variant enum onto Button variants (primary→default, danger→destructive) so Approve/Reject get hierarchy (P1-5) - Resolved lookup keys render as "Owner", not "Owner Id" (P2) - New approvalsInbox keys (flowOrigin, sort*) in all ten locales Refs objectui#2762 Co-Authored-By: Claude Fable 5 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_01Cu48mLFUdRBmMh8Z8R3CVz
|
The latest updates on your projects. Learn more about Vercel for GitHub. |
Contributor
✅ Console Performance Budget
📦 Bundle Size Report
Size Limits
|
os-zhuang
marked this pull request as ready for review
July 24, 2026 14:17
This was referenced Jul 24, 2026
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary
Second pass on #2762 — the remaining P1 items plus a P2 label polish. (P0-1/P0-2/P1-1 and the P0-3 UI half shipped in #2789; this builds on that.)
payload), preferring the server-formattedpayload_displayvalue (currency, etc.) and falling back to a localized number. A sort control adds "Oldest first" and "Amount (high→low)" next to the default newest-first. Detection is deliberately narrow (amount|total|price|value|cost|sum|budget|salary|fee|revenue|balance+ CJK equivalents, numeric value only) so it surfaces the decision figure, not every number.submitter_id, or aflow:/system:actor) now read "Flow-initiated" with a workflow icon instead of a bare person icon + "—", consistently in the desktop table, mobile card, and drawer summary.DeclaredActionsBarnow maps the spec actionvariantenum onto the Button variants:primary→ filled default,danger→destructive(the two names the enum and the Button component spell differently); the rest pass through. Once@objectstack/plugin-approvalsdeclaresapproval_approve: primary/approval_reject: danger(companion PR), the drawer's Approve stands out and Reject reads as destructive — matching the mobile card. Covered by a new test.owner_idrenders as "Owner", not the awkward "Owner Id", in the drawer summary.New
approvalsInboxkeys (flowOrigin,sortBy/sortRecent/sortOldest/sortAmount) added to all ten locales.Verification
turbo buildgreen for@object-ui/app-shell,@object-ui/i18n,@object-ui/consoleDeclaredActionsBar.test.tsx8/8 (incl. new variant-mapping case) ·packages/i18nsuites green (114 total)eslinton the touched files: no new errors or warnings (ApprovalsInboxPagestays at the same 9 pre-existingreact-hookserrors as main — allDate.now()-in-render /getIcon, untouched)app-shell,i18n)Companion: objectstack-ai/objectstack (P1-5 framework half — declares the
varianton the approve/reject actions).Refs #2762 (P1-2, P1-3, P1-4, P1-5, and the P2 label item)
🤖 Generated with Claude Code
https://claude.ai/code/session_01Cu48mLFUdRBmMh8Z8R3CVz
Generated by Claude Code