feat: unify suggestions and PRs views with mode-aware labels#149
feat: unify suggestions and PRs views with mode-aware labels#149R-Hart80 wants to merge 2 commits intoCatholicOS:devfrom
Conversation
In standard mode, the PR list and actions now use suggestion-friendly language: tabs show "Accepted"/"Rejected" instead of "Merged"/"Closed", action buttons say "Accept"/"Reject" instead of "Merge"/"Close", and confirmation dialogs and empty-state messages follow suit. The editor toolbar now shows a single mode-aware button: - Standard mode (isSuggestionMode): "My Suggestions" in amber, linking to /pull-requests with suggestion labels - Developer mode: "My Pull Requests" with open-PR count badge The pull-requests list page and detail breadcrumb also adapt their titles and icons based on the active editor mode. Closes CatholicOS#65 Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
|
Warning Rate limit exceeded
Your organization is not enrolled in usage-based pricing. Contact your admin to enable usage-based pricing to continue reviews beyond the rate limit, or try again in 22 minutes and 55 seconds. ⌛ How to resolve this issue?After the wait time has elapsed, a review can be triggered using the We recommend that you space out your commits to avoid hitting the rate limit. 🚦 How do rate limits work?CodeRabbit enforces hourly rate limits for each developer per organization. Our paid plans have higher rate limits than the trial, open-source and free plans. In all cases, we re-allow further reviews after a brief timeout. Please see our FAQ for further information. ℹ️ Review info⚙️ Run configurationConfiguration used: defaults Review profile: CHILL Plan: Pro Run ID: 📒 Files selected for processing (7)
✨ Finishing Touches🧪 Generate unit tests (beta)
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 |
The previous condition (isSuggestionMode || editorMode === "developer") hid the button for owners/admins in standard mode, since their isSuggestionMode is false (they edit directly, not via suggestions). The button should always be visible — only its label and style adapt to editorMode (standard → "My Suggestions" amber/lightbulb, developer → "My Pull Requests" with open-PR count badge). Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
Closes #65
Summary
PRList: addsmodeprop — tabs show Accepted/Rejected instead of Merged/Closed in standard mode; count says "suggestion(s)"; empty-state messages adaptPRListItem: addsmodeprop — status badges and date text ("accepted X ago") adaptPRActions: addsmodeprop — Merge→Accept, Close→Reject; confirmation dialog titles/descriptions adaptPRDetail: addsmodeprop, passes it through toPRActionspull-requests/page.tsx: readseditorModefrom store; title becomes My Suggestions + amber Lightbulb icon in standard mode; "New Pull Request" → New Suggestion; passesmodetoPRListpull-requests/[prNumber]/page.tsx: breadcrumb shows My Suggestions in standard mode; passesmodetoPRDetaileditor/page.tsx: the separate "My Suggestions" and "PRs" toolbar buttons are replaced by a single mode-aware button:/pull-requests/pull-requestsTest plan
🤖 Generated with Claude Code