Skip to content

Add import file to mobile team view#207

Merged
sarahentzel merged 4 commits intodevelopfrom
TT-7062
Feb 12, 2026
Merged

Add import file to mobile team view#207
sarahentzel merged 4 commits intodevelopfrom
TT-7062

Conversation

@sarahentzel
Copy link
Collaborator

centralize team code

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 centralizes team-related UI logic by extracting shared “team actions” behavior into a reusable hook and dialog component, and exposes the import dialog from the mobile team switch view.

Changes:

  • Refactor SwitchTeams floating actions to use a shared useTeamActions hook and always render ImportTab when opened.
  • Refactor TeamActions to reuse the same useTeamActions hook and extract the shared content creator UI into SharedContentCreatorDialog.
  • Add new shared modules: useTeamActions and SharedContentCreatorDialog.

Reviewed changes

Copilot reviewed 4 out of 4 changed files in this pull request and generated 4 comments.

File Description
src/renderer/src/routes/SwitchTeams.tsx Uses shared team actions hook; makes import dialog accessible from this view; replaces inline shared-content dialog with extracted component.
src/renderer/src/components/Team/useTeamActions.ts New hook consolidating Add Team / Import / Shared Content Creator state + handlers.
src/renderer/src/components/Team/TeamActions.tsx Refactored to consume useTeamActions and SharedContentCreatorDialog.
src/renderer/src/components/Team/SharedContentCreatorDialog.tsx New reusable dialog component for Shared Content Creator flow.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

@sarahentzel sarahentzel merged commit 19471c2 into develop Feb 12, 2026
1 check passed
@sarahentzel sarahentzel deleted the TT-7062 branch February 12, 2026 22:18
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