Skip to content

Added prevents of deletion of singular parent assets #16380

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

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

Conversation

Godmartinz
Copy link
Member

This prevents deletion of assets that are not checked out, but have assets checked out to them. Also changes the hover text to point out that the asset has other assets checked out to it. check out the image below.

The asset count is eager loaded in the api call so queries stay at a reasonable 25.

image

Copy link

what-the-diff bot commented Feb 27, 2025

PR Summary

  • Enhanced Asset Information
    The 'Assets' section on our platform now delivers additional information by showcasing linked 'assets'. This makes it more comprehensive for users who wish to view related assets.

  • Inclusion of Sub-Assets Count
    A new feature counts the number of sub-assets for any given asset. This provides a swift summary and keeps the user abreast of the hierarchy of the inventory.

  • Improved Delete Logic
    We've improved how our system determines whether an asset can be safely deleted. Now, an asset cannot be erased if it has any sub-assets linked to it, reinforcing data integrity and preventing accidental deletion of nestled data.

  • Upgraded Error Messaging
    Our system now uses an updated language string in the event of a user attempting to delete an asset that has sub-assets attached. This ensures clear and unambiguous communication regarding why the deletion attempt was not successful.

  • Updated Tooltip Messaging for Deletion
    The tooltips for the 'delete' button now accurately reflect new rules around the deletion of an asset based on its linked sub-assets. This serves as an intuitive guide for users.

@Godmartinz Godmartinz changed the title Added prevents deletion of singular parent assets Added prevents of deletion of singular parent assets Feb 27, 2025
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