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

Added asset name to bulk asset edit #15291

Merged
merged 2 commits into from
Aug 14, 2024
Merged

Conversation

snipe
Copy link
Owner

@snipe snipe commented Aug 14, 2024

What it says on the tin :)

Copy link

what-the-diff bot commented Aug 14, 2024

PR Summary

  • Added name field assessment in 'BulkAssetsController.php'
    The pull request introduces a mechanism to ensure that the name field is accounted for while handling bulk assets. This helps in maintaining data accuracy and integrity.

  • Conditional updating of 'name' field in 'BulkAssetsController.php'
    The code update also furnishes the functionality to conditionally add or modify the 'name' field in the update array. Apart from this, it also introduces the option to invalidate it (set to null) when necessary. This provides more flexibility and control while managing bulk assets.

  • Updated user interface in 'bulk.blade.php'
    The PR includes an improvement in the user interface, by adding a new form field for name, along with a checkbox allowing users to set it to null. This offers users a more interactive and easy way to manage the 'name' field of bulk assets.

@snipe snipe merged commit e544007 into develop Aug 14, 2024
8 checks passed
@snipe snipe deleted the features/bulk_update_asset_name branch August 14, 2024 09:02
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.

1 participant