Remove unused plugin list components and filters#5915
Conversation
|
|
The latest updates on your projects. Learn more about Vercel for GitHub. 1 Skipped Deployment
|
There was a problem hiding this comment.
Pull Request Overview
This PR removes unused plugin list components and filtering functionality from the dashboard, focusing on cleaning up dead code related to the plugin management UI.
- Removes plugin list UI components including
PluginsList,PluginsListPage, and related table components - Eliminates plugin filtering logic, tests, and associated utilities
- Changes plugin URL helpers from exported to internal constants to prevent external usage
Reviewed Changes
Copilot reviewed 33 out of 33 changed files in this pull request and generated no comments.
Show a summary per file
| File | Description |
|---|---|
| src/utils/maps.ts | Removes unused mapPagesToChoices function and related imports |
| src/plugins/views/PluginList/ | Deletes entire directory containing filtering logic, sorting, and main PluginList component |
| src/plugins/urls.ts | Changes pluginListPath and pluginPath from exported to internal constants |
| src/plugins/plugins-with-app-replacements.ts | Removes deprecated function for identifying plugins with app replacements |
| src/plugins/components/PluginsListPage/ | Removes plugin list page component and related filtering infrastructure |
| src/plugins/components/PluginsList/ | Deletes plugin list table components and availability cells |
| src/misc.ts | Removes unused findById utility function |
| src/icons/Miscellaneous.tsx | Removes deprecated Material-UI icon component |
| src/hooks/useChannelsSearchWithLoadMore.ts | Deletes unused hook for channel search with pagination |
| src/extensions/components/EmptyListState/EmptyInstalledList.tsx | Removes empty state component |
| src/custom-apps/views/CustomAppCreate/ | Deletes custom app creation view and messages |
| src/custom-apps/utils.tsx | Removes unused buildEventsMap function |
| src/custom-apps/fixtures.ts | Deletes test fixtures |
| src/components/SearchBar/ | Removes entire SearchBar component directory |
| src/components/FilterBar/ | Deletes FilterBar component and related functionality |
| src/apps/messages.ts | Removes unused app activation/deactivation messages |
| locale/defaultMessages.json | Cleans up unused translation keys |
Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.
Codecov Report✅ All modified and coverable lines are covered by tests. Additional details and impacted files@@ Coverage Diff @@
## main #5915 +/- ##
==========================================
+ Coverage 39.22% 39.35% +0.13%
==========================================
Files 2671 2649 -22
Lines 42277 42024 -253
Branches 9543 9481 -62
==========================================
- Hits 16582 16540 -42
+ Misses 25668 25457 -211
Partials 27 27 ☔ View full report in Codecov by Sentry. 🚀 New features to boost your workflow:
|
Summary
This PR removes dead code related to the plugin list UI and filtering functionality:
PluginsList,PluginsListPage, and related table components that are no longer usedPluginListview with filtering logic, tests, and snapshotsFilterBarandSearchBarcomponents that were only used by pluginsuseChannelsSearchWithLoadMorehook, custom app fixtures, and utility functionspluginListPathandpluginPathfrom exported to internal constants insrc/plugins/urls.ts:16andsrc/plugins/urls.ts:43🤖 Generated with Claude Code