fix: update bulk action prefix to myparcelcom#1559
fix: update bulk action prefix to myparcelcom#1559GravendeelJochem wants to merge 1 commit intomainfrom
Conversation
❌ 1 Tests Failed:
View the full list of 1 ❄️ flaky test(s)
To view more test analytics, go to the Test Analytics Dashboard |
There was a problem hiding this comment.
Pull request overview
Updates the WooCommerce admin bulk-action listener to recognize the new plugin/app prefix (myparcelcom), aligning the frontend hook with the bulk action identifiers used in the admin UI.
Changes:
- Update the bulk action prefix from
myparcelnltomyparcelcomin the admin bulk-action listener.
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
| import {AdminAction, useActionStore} from '@myparcel-dev/pdk-admin'; | ||
|
|
||
| const BULK_ACTION_PREFIX = 'myparcelnl'; | ||
| const BULK_ACTION_PREFIX = 'myparcelcom'; |
There was a problem hiding this comment.
The bulk-action value prefix is generated server-side using Pdk::getAppInfo()->name (see PdkOrderListHooks::registerBulkActions()), so hardcoding 'myparcelcom' here can get out of sync if the app name ever changes. Consider deriving the prefix from useAppInfo().name so the JS listener always matches the PHP-registered bulk actions.
INT-1407