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
A comprehensive cleanup of dead code across the dashboard application, focusing on removing unused functions, components, and utilities primarily in the plugins, filtering, and channel management modules.
- Removed entire plugin management system including views, components, and utilities
- Cleaned up unused filter helper functions and autocomplete field implementations
- Removed unused channel and discount management utilities
Reviewed Changes
Copilot reviewed 30 out of 30 changed files in this pull request and generated no comments.
Show a summary per file
| File | Description |
|---|---|
| src/utils/filters/fields.ts | Removed unused createAutocompleteField function and related imports |
| src/plugins/views/utils.ts | Deleted entire utility file containing plugin helper functions |
| src/plugins/views/PluginsDetails.tsx | Removed complete plugin details view component |
| src/plugins/views/PluginDetails.test.ts | Deleted test file for removed plugin details functionality |
| src/plugins/utils.ts | Removed plugin utility functions |
| src/plugins/urls.ts | Cleaned up unused URL constants and functions |
| src/plugins/fixtures.ts | Deleted plugin test fixtures |
| src/plugins/components/... | Removed multiple plugin component files |
| src/modeling/views/PageList/sort.ts | Removed unused filter function |
| src/modeling/views/PageList/filters.ts | Cleaned up unused filter creation utilities |
| src/giftCards/GiftCardsList/filters.ts | Removed unused filter structure and variable functions |
| src/discounts/views/SaleList/sort.ts | Removed unused sort utilities |
| src/discounts/handlers.ts | Deleted unused sale channel handler function |
| src/discounts/components/... | Cleaned up unused default exports |
| src/custom-apps/utils.tsx | Removed unused GraphQL introspection query |
| src/custom-apps/urls.ts | Cleaned up unused URL utilities |
| src/custom-apps/components/WebhookDetailsPage/WebhookDetailsPage.tsx | Changed export from named to default |
| src/components/Filter/index.ts | Removed unused default export |
| src/channels/utils.ts | Removed unused sale-related channel utilities |
| src/apps/urls.ts | Cleaned up unused URL path functions |
Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.
There was a problem hiding this comment.
Pull Request Overview
Copilot reviewed 79 out of 79 changed files in this pull request and generated 3 comments.
Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.
src/extensions/components/WebhookDetailsPage/components/WebhookHeaders/WebhookHeaders.test.tsx
Show resolved
Hide resolved
Codecov Report✅ All modified and coverable lines are covered by tests. Additional details and impacted files@@ Coverage Diff @@
## main #5918 +/- ##
==========================================
- Coverage 39.69% 39.64% -0.05%
==========================================
Files 2624 2582 -42
Lines 41667 40888 -779
Branches 9398 9219 -179
==========================================
- Hits 16538 16210 -328
+ Misses 25102 23444 -1658
- Partials 27 1234 +1207 ☔ View full report in Codecov by Sentry. 🚀 New features to boost your workflow:
|
Summary
This PR removes a significant batch of unused code across multiple modules, improving codebase maintainability and reducing bundle size. The changes primarily remove:
custom-apps/that have been superseded by implementations inextensions/Key Changes
Webhook Management (
src/custom-apps/components/)WebhookDetailsPage,WebhookEvents,WebhookHeaders,WebhookInfo,WebhookStatus, andWebhookSubscriptionQuerycomponents@dashboard/extensions/insteadPlugin System (
src/plugins/)PluginAuthorization,PluginDetailsChannelsCard,PluginInfo,PluginSecretFieldDialog,PluginSettings,PluginsDetailsPagePluginDetails.test.tsandPluginsDetails.tsxviewDiscounts (
src/discounts/)SaleDetailsPage,SaleInfo,SaleListPage,SaleType,SaleValueOther Modules
ControlledSwitchcomponent fromsrc/components/apps,channels, andmodelingmodulesgiftCardsandmodelingmodulesImpact
Testing
extensions/🤖 Generated with Claude Code