New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
fix(content-manager): delete and unpublish bulk actions migration to v5 #20213
fix(content-manager): delete and unpublish bulk actions migration to v5 #20213
Conversation
The latest updates on your projects. Learn more about Vercel for Git ↗︎
|
Size Change: +13 B (0%) Total Size: 2.58 MB
ℹ️ View Unchanged
|
packages/core/content-manager/admin/src/pages/ListView/components/BulkActions/PublishAction.tsx
Outdated
Show resolved
Hide resolved
packages/core/content-manager/admin/src/pages/ListView/components/BulkActions/Actions.tsx
Outdated
Show resolved
Hide resolved
params: { | ||
locale: isDeleteAllLocale ? '*' : currentLocale, | ||
}, |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Look at how we do other actions, lets pass all the plugin params and build them correctly imo.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
but BE expects just locale, can you please give an example?
packages/core/content-manager/admin/src/pages/ListView/components/BulkActions/Actions.tsx
Outdated
Show resolved
Hide resolved
packages/core/content-manager/admin/src/pages/ListView/components/BulkActions/Actions.tsx
Outdated
Show resolved
Hide resolved
packages/core/content-manager/admin/src/pages/ListView/components/BulkActions/Actions.tsx
Outdated
Show resolved
Hide resolved
packages/core/content-manager/admin/src/pages/ListView/components/BulkActions/Actions.tsx
Show resolved
Hide resolved
const { unpublishMany: bulkUnpublishAction } = useDocumentActions(); | ||
|
||
const handleConfirmBulkUnpublish = async () => { | ||
const data = await bulkUnpublishAction({ documentIds, model }); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
We probably need to be sending all the params here otherwise we'll just unpublish the default locale.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
this isn't expecting any params so its default locale only now
What does it do?
Describe the technical changes you did.
Why is it needed?
Describe the issue you are solving.
How to test it?
Provide information about the environment and the path to verify the behaviour.
Related issue(s)/PR(s)
Let us know if this is related to any issue/pull request