Skip to content

Conversation

@sarahentzel
Copy link
Collaborator

No description provided.

Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

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 useEffect call formatting in ProjectMenu and refactors the settings menu condition for readability without changing behavior.
  • Integrates useDataChanges into ProjectCard so 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();
Copy link

Copilot AI Feb 4, 2026

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

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.

Suggested change
copyComplete();
dispatch(copyComplete() as any);

Copilot uses AI. Check for mistakes.
@sarahentzel sarahentzel merged commit 2a958d5 into develop Feb 4, 2026
1 check passed
@sarahentzel sarahentzel deleted the TT-7063 branch February 4, 2026 18:48
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant