cleanup: remove integrations settings placeholder #334

Merged

barrettruth merged 1 commit from cleanup/remove-integrations-settings-route into main

2026-05-11 19:50:53 +00:00

barrettruth commented

2026-05-11 19:49:37 +00:00

Owner

Problem

The integrations settings route is an empty placeholder that points users back to calendar settings, so it makes the settings surface look larger than the live product.

Solution

Remove the empty integrations settings page and component.
Drop integrations from SETTINGS_SECTIONS so the settings nav and numeric shortcuts skip it.
Update settings/keymap tests around the final nav order and remove the placeholder component test.

Verification

nix develop --command pnpm exec vitest run tests/lib/settings-navigation.test.ts tests/lib/keymap-defs.test.ts tests/lib/settings-primitives.test.ts --reporter=verbose
nix develop --command just lint

Closes #313

## Problem The integrations settings route is an empty placeholder that points users back to calendar settings, so it makes the settings surface look larger than the live product. ## Solution - Remove the empty integrations settings page and component. - Drop integrations from SETTINGS_SECTIONS so the settings nav and numeric shortcuts skip it. - Update settings/keymap tests around the final nav order and remove the placeholder component test. ## Verification - nix develop --command pnpm exec vitest run tests/lib/settings-navigation.test.ts tests/lib/keymap-defs.test.ts tests/lib/settings-primitives.test.ts --reporter=verbose - nix develop --command just lint Closes #313

barrettruth added 1 commit

2026-05-11 19:49:37 +00:00

cleanup: remove integrations settings placeholder

quality / Test (pull_request) Successful in 17s

Details

quality / Lint (pull_request) Successful in 20s

Details

quality / Build (pull_request) Successful in 42s

Details

8daf1fcc33