cleanup: remove integrations settings route #371

Merged

barrettruth merged 1 commit from issue-313-remove-integrations-settings-route into main

2026-05-11 21:23:11 +00:00

barrettruth commented

2026-05-11 21:08:37 +00:00

Owner

Problem

The settings integrations page is a placeholder that points users back to calendar settings, so settings still advertises an empty surface.

Solution

Remove the direct and intercepted /settings/integrations pages.
Remove IntegrationsSection and its placeholder test.
Remove the integrations nav section and update settings navigation/keymap tests for the remaining account, calendar, and preferences order.
Keep /api/settings/integrations* routes and the calendar provider settings flow intact.

Verification

nix develop --command sh -c 'pnpm install --frozen-lockfile && pnpm vitest run tests/lib/settings-navigation.test.ts tests/lib/keymap-defs.test.ts tests/lib/settings-primitives.test.ts tests/lib/calendar-settings-section.test.ts tests/api/settings-integrations.test.ts tests/api/integrations-test-route.test.ts --reporter=verbose'
nix develop --command just lint
nix develop --command just test
nix develop --command just build

Closes #313

## Problem The settings integrations page is a placeholder that points users back to calendar settings, so settings still advertises an empty surface. ## Solution - Remove the direct and intercepted `/settings/integrations` pages. - Remove `IntegrationsSection` and its placeholder test. - Remove the integrations nav section and update settings navigation/keymap tests for the remaining account, calendar, and preferences order. - Keep `/api/settings/integrations*` routes and the calendar provider settings flow intact. ## Verification - `nix develop --command sh -c 'pnpm install --frozen-lockfile && pnpm vitest run tests/lib/settings-navigation.test.ts tests/lib/keymap-defs.test.ts tests/lib/settings-primitives.test.ts tests/lib/calendar-settings-section.test.ts tests/api/settings-integrations.test.ts tests/api/integrations-test-route.test.ts --reporter=verbose'` - `nix develop --command just lint` - `nix develop --command just test` - `nix develop --command just build` Closes #313

barrettruth added 1 commit

2026-05-11 21:08:37 +00:00

cleanup: remove integrations settings route

quality / Test (pull_request) Successful in 16s

Details

quality / Lint (pull_request) Successful in 21s

Details

quality / Build (pull_request) Successful in 43s

Details

8f85e9df72