Skip to content

Improves modal copy when deleting page assets leads to package deletion

What does this MR do and why?

When the user deletes all or last of the package assets from a package, it may not be obvious to the user that delete would be deleted.

This MR updates the copy to be more specific in such cases.

Screenshots or screen recordings

header header
all files (before) Screen_Shot_2022-09-29_at_11.24.12_pm
all files Screen_Shot_2022-09-30_at_12.26.43_am
last file (before) Screen_Shot_2022-09-29_at_11.24.33_pm
last file Screen_Shot_2022-09-30_at_12.26.18_am

Since the screenshot was taken before #349515 (closed) got merged, the button text still says Permanently delete package. Instead the modal should now say Permanently delete instead as mentioned

How to set up and validate locally

  1. Follow instructions here to publish a new package
  2. Publish couple of packages. One with only 1 file and another with multiple files
  3. Visit package details page
  4. Attempting to delete last package file should show appropriate message as above
  5. Attempting to delete all package files should show appropriate message as above

MR acceptance checklist

This checklist encourages us to confirm any changes have been analyzed to reduce risks in quality, performance, reliability, security, and maintainability.

Closes #368908 (closed)

Edited by Rahul Chanila

Merge request reports

Loading