Protected containers: Fix error message in container overview on protected container
-
Please check this box if this contribution uses AI-generated content (including content generated by GitLab Duo features) as outlined in the GitLab DCO & CLA. As a benefit of being a GitLab Community Contributor, you receive complimentary access to GitLab Duo.
What does this MR do and why?
During the review for the implemnentation of the container deletion protection for GraphQL, it was noticed that the displayed error message was missing when a user attempted to delete a protected container image, see !183545 (comment 2640873119).
The frontend error handling was not correctly processing error arrays that contain string values instead of objects with message properties. This update ensures both string and object error formats are properly displayed to users when container image deletion fails.
Changelog: fixed
References
Screenshots or screen recordings
| Before | After |
|---|---|
![]() |
![]() |
How to set up and validate locally
- Ensure you have the feature flag :container_registry_protected_containers_delete enabled.
Feature.enabled(:container_registry_protected_containers_delete)
- Push a container image to the container registry of your GDK to the project http://gdk.test:3000/flightjs/Flight
- Go to http://gdk.test:3000/flightjs/Flight/-/settings/packages_and_registries and create (or alter) a container protection rule to protect the recently pushed container image
- Go to http://gdk.test:3000/flightjs/Flight/container_registry and attempt to delete an existing container image that should be protected
MR acceptance checklist
Please evaluate this MR against the MR acceptance checklist. It helps you analyze changes to reduce risks in quality, performance, reliability, security, and maintainability.
MR Checklist (@gerardo-navarro)
-
Changelog entry added, if necessary -
Documentation created/updated via this MR -
Documentation reviewed by technical writer or follow-up review issue created -
Tests added for this feature/bug -
Tested in all supported browsers -
Conforms to the code review guidelines -
Conforms to the style guides -
Conforms to the javascript style guides -
Conforms to the database guides -
Conforms to the merge request performance guidelines -
Wait for Protected containers: Integrate delete protecti... (!183545 - merged)
Related to #406797
Edited by Gerardo Navarro

