Dev to Main Sync #287
Dev to Main Sync #287
Conversation
…es (#286) * refactor(team): update audit log action for POC changes in team updates * test(team): add unit test for audit log creation on POC change during team update
|
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 |
![korbit-ai[bot]](https://avatars.githubusercontent.com/in/322216?s=60&v=4){kind=small}
There was a problem hiding this comment.
I've completed my review and didn't find any issues... but I did find this horse.
.''
._.-.___.' (`\
//( ( `'
'/ )\ ).__. )
' <' `\ ._/'\
` \ \Files scanned
| File Path | Reviewed |
|---|---|
| todo/services/team_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: 21 Oct 2025
Developer Name: @AnujChhikara
PRs going in Sync
Description
Documentation Updated?
Under Feature Flag
Database Changes
Breaking Changes
Development Tested?
Screenshots
Staging Proof
Description by Korbit AI
What change is being made?
Update team update flow to log POC changes instead of a generic team update, and add tests to verify the Poc Changed audit log is created with correct fields.
Why are these changes being made?
To provide more precise auditing for changes to the Point of Contact on a team, improving traceability. If there are any edge cases or future audit actions, this lays groundwork for more granular logs.