fix: Pending user input displays with "Pending Approval" status#688
fix: Pending user input displays with "Pending Approval" status#688cameronmolen wants to merge 1 commit intopingdotgg:mainfrom
Conversation
- move thread status pill derivation into `Sidebar.logic.ts` - add unit tests for status priority and unseen completion behavior - show `Pending Approval` when either approvals or user input are pending
|
Important Review skippedAuto reviews are disabled on this repository. Please check the settings in the CodeRabbit UI or the ⚙️ Run configurationConfiguration used: Repository UI Review profile: CHILL Plan: Pro Run ID: You can disable this status message by setting the Use the checkbox below for a quick retry:
✨ Finishing Touches🧪 Generate unit tests (beta)
Tip Try Coding Plans. Let us write the prompt for your AI agent so you can ship faster (with fewer bugs). Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. Comment |
|
+1 for this |
|
In my opinion, "input needed" might be better than "pending approval" |
What Changed
Pending Approvalwhen either approvals or user input are pendingSidebar.logic.tsWhy
Issue #442:
Cause:
pendingUserInputwas not being considered for the "Pending Approval" status.UI Changes
Before
before.mov
After
after.mov
Checklist
Note
Treat pending user-input as "Pending Approval" and centralize status pill resolution via
Sidebar.logic.resolveThreadStatusPillin Sidebar.tsxMove status pill logic into
Sidebar.logic.resolveThreadStatusPilland addhasUnseenCompletion; updateSidebar.tsxto use the resolver and derive pending user input; add tests in Sidebar.logic.test.ts.📍Where to Start
Start with
resolveThreadStatusPillandhasUnseenCompletionin Sidebar.logic.ts.Macroscope summarized b2259aa.