Skip to content

Conversation

@AnujChhikara
Copy link
Contributor

@AnujChhikara AnujChhikara commented Jan 18, 2026

Date: 18 Jan 2026

Developer Name: @AnujChhikara


Issue Ticket Number

Tech Doc Link

  • NA

Business Doc Link

  • NA

Description

Since My Site has been deprecated, users can now update their tasks only via the Status Site. Requiring a dev flag for task updates is no longer valid behavior. The functionality should work without any environment-specific flags.

Documentation Updated?

  • Yes
  • No

Under Feature Flag

  • Yes
  • No

Database Changes

  • Yes
  • No

Breaking Changes

  • Yes
  • No

Development Tested?

  • Yes
  • No

Screenshots

Screenshot 1
Screen.Recording.2026-01-18.at.3.04.11.PM.mov

Test Coverage

Screenshot 1

Additional Notes

@vercel
Copy link

vercel bot commented Jan 18, 2026

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Review Updated (UTC)
status-app Ready Ready Preview, Comment Jan 18, 2026 9:38am

@coderabbitai
Copy link

coderabbitai bot commented Jan 18, 2026

Walkthrough

This change simplifies the task update logic in TaskStatusEditMode by removing the dev mode requirement from the conditional branch. Now updateSelfTask is invoked based solely on ownership status (isSelfTask), rather than the conjunction of dev mode and ownership.

Changes

Cohort / File(s) Summary
Task Status Edit Mode
src/components/tasks/card/TaskStatusEditMode.tsx
Removed dev mode check from task update conditional; now uses updateSelfTask based only on isSelfTask status

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~3 minutes

Possibly related PRs

Suggested reviewers

  • iamitprakash

Poem

🐰 Dev mode chains are now released,
Self-task updates pass the test,
Simpler paths, less branching mess,
Ownership rules—nothing less! ✨

🚥 Pre-merge checks | ✅ 3
✅ Passed checks (3 passed)
Check name Status Explanation
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.
Description check ✅ Passed The PR description clearly explains the removal of the dev mode check, referencing issue #1388 and providing context about My Site deprecation and why the environment flag is no longer needed.
Title check ✅ Passed The title clearly and specifically describes the main change: removing an unnecessary dev mode check for self task updates.

✏️ Tip: You can configure your own custom pre-merge checks in the settings.

✨ Finishing touches
  • 📝 Generate docstrings

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.

❤️ Share

Comment @coderabbitai help to get the list of available commands and usage tips.

@AnujChhikara AnujChhikara changed the title fix: remove unnecessary dev mode check for self task updates HOTFIX(#1388): remove unnecessary dev mode check for self task updates Jan 18, 2026
@yesyash yesyash merged commit a131abb into develop Jan 18, 2026
4 checks passed
@yesyash yesyash deleted the anuj/hotfix-task-status-udpate-fix branch January 18, 2026 09:46
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.

3 participants