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

Fixed #15320 - add deployable status to bulk checkout #15823

Open
wants to merge 1 commit into
base: develop
Choose a base branch
from

Conversation

snipe
Copy link
Owner

@snipe snipe commented Nov 14, 2024

This adds a status dropdown to the bulk checkout page, which presents all statuses that are currently in a deployable meta status.

I'm not sure if this is exactly the right solution though, since multiple assets might have different status labels, and this would overwrite them. If I apply a placeholder to that list, it fails validation because the status_id is being passed blank, which is not permitted via the AssetCheckoutRequest.

Fixes #15320

Signed-off-by: snipe <snipe@snipe.net>
Copy link

what-the-diff bot commented Nov 14, 2024

PR Summary

  • Enhancement to the 'showCheckout' method
    The 'showCheckout' function has been updated to now share a 'statusLabel_list' with the view. This will offer the user more contextual information when checking out.

  • Augmented functionality in 'storeCheckout'
    There's a newly added feature to the 'storeCheckout' function. Now, if a 'status_id' is provided in the request, the function is able to set it. This offers more control for whoever makes the request.

  • Updates to the 'bulk-checkout' view
    The 'bulk-checkout' view now includes a new form group made for selecting a status. Error handling for 'status_id' is also implemented to ensure resilience and proper user guidance. Users now can choose the status when doing bulk operations, improving overall user experience.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant