Conversation
There was a problem hiding this comment.
Pull request overview
This PR makes small updates to the team project UI, mainly around project copying and menu behavior.
Changes:
- Cleans up the empty
useEffectcall formatting inProjectMenuand refactors the settings menu condition for readability without changing behavior. - Integrates
useDataChangesintoProjectCardso that, after a project copy completes, data changes are forced before clearing busy state and showing a completion message. - Improves the copy-project dialog by disabling the Copy button while copying or when no team has been selected.
Reviewed changes
Copilot reviewed 2 out of 2 changed files in this pull request and generated 3 comments.
| File | Description |
|---|---|
src/renderer/src/components/Team/ProjectMenu.tsx |
Minor formatting change to an empty useEffect and reformatting of the settings menu condition to a multi-line, more readable form. |
src/renderer/src/components/Team/ProjectCard.tsx |
Wires in useDataChanges for project copy completion, adjusts the copy-completion messaging, and disables the Copy button when copying or when no team is selected. |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
| @@ -211,8 +212,16 @@ export const ProjectCard = (props: IProps) => { | |||
| if (copyStatus.errStatus || copyStatus.complete) { | |||
| copyComplete(); | |||
There was a problem hiding this comment.
copyComplete is a thunk action creator (see store/importexport/actions.tsx) but here it is called directly without being dispatched, so the COPY_COMPLETE action is never sent and importexportStatus is not cleared. To ensure the Redux state is updated, this should be dispatched (e.g., via dispatch(actions.copyComplete() as any) or a similar pattern) instead of calling the thunk function directly.
| copyComplete(); | |
| dispatch(copyComplete() as any); |
No description provided.