feat(delete collections): remove collections from the tool#1271
Merged
Conversation
jpetto
force-pushed
the
HNT-1730-remove-collections
branch
3 times, most recently
from
9909904 to
a4b0cc5
Compare
mmiermans
reviewed
Feb 23, 2026
Contributor
mmiermans
left a comment
mmiermans
left a comment
There was a problem hiding this comment.
Yay! Nice work. I asked Claude to look for more collection-related code that can be removed. According to Claude, the following code/packages are unused now that collections is going away. Could you double-check and remove if so?
Files to delete
Path Why dead src/_shared/components/MarkdownPreview/No consumers src/_shared/components/TabSet/Only used by MarkdownPreview src/_shared/components/TabPanel/Only used by MarkdownPreview src/_shared/components/TabNavigation/Only used by TabSet src/_shared/components/Tab/Only used by TabSet + TabNavigation src/_shared/components/LoadMore/No consumers src/_shared/components/ScrollToTop/No consumers src/_shared/hooks/useFetchMoreResults/No consumers src/_shared/styled/StyledCardLink.tsxNo consumers src/_shared/styled/StyledChipComponent.tsxNo consumers Dead config
Remove the
slugifyblock insrc/config/index.ts(lines 47-55) — nothing readsconfig.slugifyanymore.Barrel export removals
src/_shared/components/index.ts: Remove exports forLoadMore,MarkdownPreview,ScrollToTop,Tab,TabNavigation,TabPanel,TabSet,CustomTabType
src/_shared/styled/index.ts: Remove exports forStyledCardLink,StyledChipComponentPackages to remove from
package.json
react-markdown(6.0.3) — sole consumer was MarkdownPreviewreact-beautiful-dnd(13.1.1) — sole consumer was ReorderableCollectionStoryList@types/react-beautiful-dnd(13.1.4) — type defs for the aboveslugify(1.6.6) — sole consumer was collections slug generation
| import { useNotifications } from '../'; | ||
|
|
||
| // 2026-02-23: after removing collections, this function is not currently in | ||
| // use. leaving this in the code base as we may again need this functionality. |
Contributor
There was a problem hiding this comment.
Let's delete this unused code. It should be fairly easy to recreate similar code if we ever need it again.
jpetto
force-pushed
the
HNT-1730-remove-collections
branch
3 times, most recently
from
c7213bc to
6229a27
Compare
- pocket decommission - HNT-1730 - fix lint issues - update README - add allowAds and followable to satisfy sections type checking
- delete more unused files and imports - delete unused config
jpetto
force-pushed
the
HNT-1730-remove-collections
branch
from
6229a27 to
dd04b05
Compare
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Goal
remove collections management from the admin tools. collections are being torn down as part of the larger pocket decommission.
allowAdsandfollowablewhere necessary for type checking on SectionsTodos
Reference
Tickets:
Implementation Decisions