-
Notifications
You must be signed in to change notification settings - Fork 14
Dev to Main Sync #289
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Dev to Main Sync #289
Conversation
…ic (#288) * feat(task): add team_id support in task assignment and serializer validation * refactor(task): update audit log action from 'assigned_to_user_from_team' to 'assigned_to_member'
|
Important Review skippedAuto reviews are disabled on base/target branches other than the default branch. Please check the settings in the CodeRabbit UI or the You can disable this status message by setting the ✨ 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 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Review by Korbit AI
Korbit automatically attempts to detect when you fix issues in new commits.
| Category | Issue | Status |
|---|---|---|
| Missing help_text in team_id field ▹ view | ||
| Unnecessary string strip() operation ▹ view | ||
| Missing help_text for team_id field ▹ view | ||
| Duplicate team validation query ▹ view | ||
| Missing log level in audit log creation ▹ view | ||
| Missing required team_id validation for team assignments ▹ view | ||
| Overly nested validation logic ▹ view | ||
| Complex Team ID Assignment Logic ▹ view | ||
| Generic exception instead of domain-specific exception ▹ view | ||
| Inconsistent team_id handling between validation and assignment ▹ view |
Files scanned
| File Path | Reviewed |
|---|---|
| todo/serializers/create_task_serializer.py | ✅ |
| todo/services/task_assignment_service.py | ✅ |
| todo/services/task_service.py | ✅ |
Explore our documentation to understand the languages and file types we support and the files we ignore.
Check out our docs on how you can make Korbit work best for you and your team.
Date: 4 Nov 2025
Developer Name: @AnujChhikara
Issue Resolved
PRs going in Sync
Description
Documentation Updated?
Under Feature Flag
Database Changes
Breaking Changes
Development Tested?
Screenshots
Staging Proof
Screen.Recording.2025-11-04.at.12.01.09.AM.mov
Description by Korbit AI
What change is being made?
Add support for Point of Contact (POC) change activity across the codebase: extend activity types, UI rendering, and cache invalidation.
Why are these changes being made?
To surface POC changes in the team activity timeline and ensure the UI stays up-to-date after a POC update. This introduces a new activity type, corresponding UI data, and cache invalidation for the team activities feed.