Skip to content

refactor: use alias import for GeneralTab in settings-tabs#1992

Merged
peppescg merged 1 commit intomainfrom
refactor/settings-tabs-alias-import
Apr 14, 2026
Merged

refactor: use alias import for GeneralTab in settings-tabs#1992
peppescg merged 1 commit intomainfrom
refactor/settings-tabs-alias-import

Conversation

@peppescg
Copy link
Copy Markdown
Collaborator

@peppescg peppescg commented Apr 14, 2026

Summary

  • Change the GeneralTab import in settings-tabs.tsx from relative (./general-tab) to alias-based (@/common/components/settings/tabs/general-tab)
  • This allows enterprise overlays to swap the GeneralTab component via Vite alias without having to duplicate the entire settings-tabs.tsx file

Test plan

  • Verify settings page loads correctly with all tabs
  • Verify General tab renders theme, auto-launch, telemetry, quit confirmation fields
  • Run pnpm run type-check and pnpm run lint

🤖 Generated with Claude Code

Closes #2017

Change the GeneralTab import from relative (`./general-tab`) to alias-based
(`@/common/components/settings/tabs/general-tab`) to allow enterprise overlays
to swap the component via Vite alias without duplicating settings-tabs.

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
Copilot AI review requested due to automatic review settings April 14, 2026 11:37
@peppescg peppescg self-assigned this Apr 14, 2026
Copy link
Copy Markdown
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

Refactors the GeneralTab import in settings tabs to use an @/ alias path so enterprise overlays can swap the General tab implementation via Vite aliasing without copying settings-tabs.tsx.

Changes:

  • Updated GeneralTab import in settings-tabs.tsx from a relative path to an alias-based path (@/...) to support overlay replacement.

Copy link
Copy Markdown
Collaborator

@samuv samuv left a comment

Choose a reason for hiding this comment

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

👍🏼

@peppescg peppescg merged commit 22f252c into main Apr 14, 2026
22 checks passed
@peppescg peppescg deleted the refactor/settings-tabs-alias-import branch April 14, 2026 11:49
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[Task] Replace import alias for general-tab

3 participants