Skip to content
This repository was archived by the owner on Aug 6, 2026. It is now read-only.

feat(folder-picker): remove a folder from recents - #2859

Merged
adboio merged 1 commit into
mainfrom
posthog-code/folder-picker-remove-recent
Jun 27, 2026
Merged

feat(folder-picker): remove a folder from recents#2859
adboio merged 1 commit into
mainfrom
posthog-code/folder-picker-remove-recent

Conversation

@richardsolomou

Copy link
Copy Markdown
Member

Problem

No way to remove a folder from the Recent list in the folder picker. The only path was the buried "Danger zone" in FolderSettingsView.

Changes

Add a hover-revealed remove button to each row in the Recent dropdown (packages/ui/src/features/folder-picker/FolderPicker.tsx). It reuses the existing useFolders().removeFolder hook. Gated behind a confirmation dialog because removeFolder is destructive: it also deletes the repo's tasks and worktrees.

How did you test this?

pnpm --filter @posthog/ui typecheck and biome lint on the file, both clean. Did not verify in the running app (the live dev instance is serving a different checkout).

Automatic notifications

  • Publish to changelog?
  • Alert Sales and Marketing teams?

Created with PostHog Code

Add a hover-revealed remove button to each row in the Recent dropdown,
gated behind a confirmation dialog since removeFolder also deletes the
repo's tasks and worktrees.
Generated-By: PostHog Code
Task-Id: efb7b41b-2cde-40ff-8bd0-13fae0932ca1
@github-actions

Copy link
Copy Markdown

React Doctor found no issues in the changed files. 🎉

Reviewed by React Doctor for commit 136def6.

@greptile-apps

Copy link
Copy Markdown
Contributor

Reviews (1): Last reviewed commit: "feat(folder-picker): remove a folder fro..." | Re-trigger Greptile

Comment threadpackages/ui/src/features/folder-picker/FolderPicker.tsx
@richardsolomou
richardsolomou requested a review from a teamJune 24, 2026 10:56
@adboio
adboio merged commit 050621f into mainJun 27, 2026
20 checks passed
@adboio
adboio deleted the posthog-code/folder-picker-remove-recent branch June 27, 2026 15:14
Sign up for freeto subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants

@richardsolomou@adboio