Add missing migration #8162
Add missing migration #8162
Conversation
WalkthroughThe recent update introduces a Django migration that modifies the Changes
Poem
Thank you for using CodeRabbit. We offer it for free to the OSS community and would appreciate your support in helping us grow. If you find it useful, would you consider giving us a shout-out on your favorite social media? TipsChatThere are 3 ways to chat with CodeRabbit:
Note: Be mindful of the bot's finite context window. It's strongly recommended to break down tasks such as reading entire modules into smaller chunks. For a focused discussion, use review comments to chat about specific files and their changes, instead of using the PR comments. CodeRabbit Commands (invoked as PR comments)
Additionally, you can add CodeRabbit Configration File (
|
|
![coderabbitai[bot]](https://avatars.githubusercontent.com/in/347564?s=60&v=4){kind=small}
There was a problem hiding this comment.
Actionable comments posted: 0
Review details
Configuration used: CodeRabbit UI
Review profile: CHILL
Files selected for processing (1)
- cvat/apps/engine/migrations/0080_alter_trackedshape_track.py (1 hunks)
Additional comments not posted (3)
cvat/apps/engine/migrations/0080_alter_trackedshape_track.py (3)
1-1: Confirm migration generation timestamp.Ensure that the timestamp matches the intended migration creation time to avoid conflicts with other migrations.
14-23: Field alteration correctness.The
trackfield in thetrackedshapemodel is updated to referenceengine.LabeledTrackwithon_delete=models.CASCADE,related_name="shapes", andrelated_query_name="shape". This change looks correct.
10-11: Dependencies check.The migration depends on
0079_alter_labeledimageattributeval_image_and_more. Ensure that this migration exists and is correctly applied before this one.
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## develop #8162 +/- ##
===========================================
- Coverage 83.43% 83.42% -0.01%
===========================================
Files 388 388
Lines 41272 41272
Branches 3852 3852
===========================================
- Hits 34435 34433 -2
- Misses 6837 6839 +2
|
<!-- Raise an issue to propose your change (https://github.com/cvat-ai/cvat/issues). It helps to avoid duplication of efforts from multiple independent contributors. Discuss your ideas with maintainers to be sure that changes will be approved and merged. Read the [Contribution guide](https://docs.cvat.ai/docs/contributing/). --> <!-- Provide a general summary of your changes in the Title above --> Depends on #8162 ### Motivation and context <!-- Why is this change required? What problem does it solve? If it fixes an open issue, please link to the issue here. Describe your changes in detail, add screenshots. --> - Added recording and reporting of the last assignee update time on the server - Added reporting of the assignee in quality reports ### How has this been tested? <!-- Please describe in detail how you tested your changes. Include details of your testing environment, and the tests you ran to see how your change affects other areas of the code, etc. --> Unit tests ### Checklist <!-- Go over all the following points, and put an `x` in all the boxes that apply. If an item isn't applicable for some reason, then ~~explicitly strikethrough~~ the whole line. If you don't do that, GitHub will show incorrect progress for the pull request. If you're unsure about any of these, don't hesitate to ask. We're here to help! --> - [ ] I submit my changes into the `develop` branch - [ ] I have created a changelog fragment <!-- see top comment in CHANGELOG.md --> - [ ] I have updated the documentation accordingly - [ ] I have added tests to cover my changes - [ ] I have linked related issues (see [GitHub docs]( https://help.github.com/en/github/managing-your-work-on-github/linking-a-pull-request-to-an-issue#linking-a-pull-request-to-an-issue-using-a-keyword)) - [ ] I have increased versions of npm packages if it is necessary ([cvat-canvas](https://github.com/cvat-ai/cvat/tree/develop/cvat-canvas#versioning), [cvat-core](https://github.com/cvat-ai/cvat/tree/develop/cvat-core#versioning), [cvat-data](https://github.com/cvat-ai/cvat/tree/develop/cvat-data#versioning) and [cvat-ui](https://github.com/cvat-ai/cvat/tree/develop/cvat-ui#versioning)) ### License - [ ] I submit _my code changes_ under the same [MIT License]( https://github.com/cvat-ai/cvat/blob/develop/LICENSE) that covers the project. Feel free to contact the maintainers if that's a concern. <!-- This is an auto-generated comment: release notes by coderabbit.ai --> ## Summary by CodeRabbit - **New Features** - Added tracking for the last assignee update time across jobs, tasks, and projects. - Introduced assignee information in quality reports. - **Bug Fixes** - Ensured accurate handling of assignee details and update times in various entities. - **Tests** - Added extensive test coverage for creating and updating assignee details in jobs, tasks, projects, and quality reports. - **Documentation** - Updated schema to include the new `assignee_updated_date` field. <!-- end of auto-generated comment: release notes by coderabbit.ai -->
Motivation and context
Follow up for #8144
How has this been tested?
Checklist
developbranch(cvat-canvas,
cvat-core,
cvat-data and
cvat-ui)
License
Feel free to contact the maintainers if that's a concern.
Summary by CodeRabbit