Closed bsekachev closed 1 week ago
[!IMPORTANT]
Review skipped
Auto incremental reviews are disabled on this repository.
Please check the settings in the CodeRabbit UI or the
.coderabbit.yaml
file in this repository. To trigger a single review, invoke the@coderabbitai review
command.You can disable this status message by setting the
reviews.review_status
tofalse
in the CodeRabbit configuration file.
The recent changes enhance the handling of job states, transitions specific actions to utilize ThunkDispatch
, and introduce new UI features. Key improvements include better type usage in asynchronous actions, updated UI handling for single shape annotation, and fixes for handling job state transitions. The changes aim to streamline the annotation process and ensure smoother interaction across the CVAT UI.
File(s) | Summary |
---|---|
cvat-ui/src/actions/annotation-actions.ts |
Updated annotation actions to use ThunkDispatch and added job state transition handling. |
cvat-ui/src/actions/cloud-storage-actions.ts |
Modified type declarations for dispatch functions to use ThunkDispatch . |
cvat-ui/src/actions/projects-actions.ts |
Updated function signatures to use ThunkDispatch . |
cvat-ui/src/actions/tasks-actions.ts |
Streamlined type definitions for asynchronous actions using ThunkDispatch . |
cvat-ui/src/actions/webhooks-actions.ts |
Adjusted getWebhooksAsync to use ThunkDispatch . |
cvat-ui/src/components/annotation-page/canvas/.../canvas-wrapper.tsx |
Included Workspace.SINGLE_SHAPE in workspace condition. |
cvat-ui/src/components/annotation-page/.../single-shape-sidebar.tsx |
Added new actions like nextFrame and updated UI elements for frame handling. |
cvat-ui/src/components/annotation-page/.../single-shape-workspace.tsx |
Integrated RemoveConfirmComponent for removing shapes. |
cvat-ui/src/components/annotation-page/.../styles.scss |
Renamed .cvat-single-shape-annotation-sidebar-skip-wrapper to .cvat-single-shape-annotation-sidebar-finish-frame-wrapper . |
cvat-ui/src/components/annotation-page/.../remove-confirm.tsx |
Enhanced useSelector hook and updated removeObjectAsync function. |
cvat-ui/src/components/annotation-page/top-bar/annotation-menu.tsx |
Refactored to dispatch actions using useDispatch . |
cvat-ui/src/components/annotation-page/top-bar/left-group.tsx |
Replaced AnnotationMenuContainer with AnnotationMenuComponent . |
cvat-ui/src/containers/.../canvas-context-menu.tsx |
Changed readonly property in OwnProps interface to an optional boolean. |
cvat-ui/src/reducers/index.ts |
Added jobUpdating property to NotificationsState interface. |
cvat-ui/src/reducers/notifications-reducer.ts |
Included jobUpdating handling for UPDATE_CURRENT_JOB_FAILED action. |
site/content/en/docs/manual/advanced/single-shape.md |
Updated hints for Single Shape annotation mode. |
tests/cypress/e2e/features/single_object_annotation.js |
Added new test cases, removed submitJob() function, and updated handling for PATCH requests. |
No diagrams needed as changes revolve around type updates and UI adjustments, which are too specific and varied.
Objective | Addressed | Explanation |
---|---|---|
Workers should be able to update job status (#7241) | β |
In code so sleek, the changes flow, Dispatch with
Thunk
, actions in tow. Transition jobs with ease anew, Single shapes in frames pursue. A worker's task made clearer now, A bunny's hop, take a bow! π°π
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?
Issues
3 New issues
0 Accepted issues
Measures
0 Security Hotspots
0.0% Coverage on New Code
0.0% Duplication on New Code
Motivation and context
Resolved #7241
How has this been tested?
Checklist
develop
branchLicense
Summary by CodeRabbit
New Features
RemoveConfirmComponent
, in the Single Shape Workspace.Bug Fixes
Documentation
Tests
Style
.cvat-single-shape-annotation-sidebar-skip-wrapper
to.cvat-single-shape-annotation-sidebar-finish-frame-wrapper
.