chore: remove dev flag from tasks dropdown#1383
Conversation
|
The latest updates on your projects. Learn more about Vercel for GitHub.
|
|
Important Review skippedAuto incremental reviews are disabled on this repository. Please check the settings in the CodeRabbit UI or the You can disable this status message by setting the WalkthroughModified a conditional in the task card component that controls the display of the task status edit mode. Removed the dev-mode requirement from the condition guarding self-task status editing UI, allowing it to appear whenever a task is marked as a self-task and is editable. Changes
Estimated code review effort🎯 2 (Simple) | ⏱️ ~10 minutes Poem
🚥 Pre-merge checks | ✅ 3✅ Passed checks (3 passed)
✏️ Tip: You can configure your own custom pre-merge checks in the settings. 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 |
There was a problem hiding this comment.
Actionable comments posted: 1
🤖 Fix all issues with AI agents
In @src/components/tasks/card/index.tsx:
- Line 630: Self-assigned tasks can render both the regular status display and
TaskStatusControl simultaneously because the TaskStatusControl condition was
loosened to (isEditable || isSelfTask) while the regular status display still
only checks !isDevMode; restore consistent gating: either change the
TaskStatusControl condition back to (isEditable || (isDevMode && isSelfTask)) so
only editors or dev-mode self-tasks see it, or change the regular status display
condition to {!isDevMode && !isSelfTask && ( ... )} so self-assigned users in
production see only TaskStatusControl; keep ExtensionStatusModal’s
dev={isDevMode} prop unchanged.
AnujChhikara
left a comment
There was a problem hiding this comment.
can you please add some description so I know the context of this change
dbb4061 to
56e18c0
Compare
Date: 11-01-26
Developer Name: @MayankBansal12
Issue Ticket Number
Description
Documentation Updated?
Under Feature Flag
Database Changes
Breaking Changes
Development Tested?
Screenshots
demo
Screencast.from.2026-01-11.01-23-49.mp4