Skip to content
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

Feature: Document entity bulk action "move to" #2122

Merged
merged 2 commits into from
Jul 17, 2024

Conversation

leekelleher
Copy link
Member

@leekelleher leekelleher commented Jul 17, 2024

Description

Implements the Document collection bulk move feature, (as a Entity Bulk Action moveTo kind).

Types of changes

  • New feature (non-breaking change which adds functionality)

How to test?

  • Ensure that "Allow bulk move" is enabled in the Collection (data-type) configuration.
  • In a document that has a Collection enabled, select 1 or more document items.
  • In the Collections actions toolbar (at the bottom), select the "Move to" action button.
  • A document picker modal should appear to select the destination/target of the items, select an item.
    • If an item is selected where child document items are not allowed, an error will be displayed.
    • If an item is selected where child document items are allowed, then the items will be moved, a success notification should be displayed and the Document Collection UI should be automatically refreshed.
  • Go to the destination/target folder and check that the document items are there.

Copy link
Collaborator

@iOvergaard iOvergaard left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@leekelleher leekelleher marked this pull request as ready for review July 17, 2024 14:17
@leekelleher leekelleher merged commit 008eb26 into main Jul 17, 2024
7 checks passed
@leekelleher leekelleher deleted the v14/feature/document-entity-bulk-action-move-to branch July 17, 2024 14:30
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants