winterhazel commented on PR #8833: URL: https://github.com/apache/cloudstack/pull/8833#issuecomment-2135773444
Hey @lucas-a-martins @DaanHoogland, sorry for the delay. Looking back at this PR, I think it would be better to implement the link validation in both pages by including the volume's state in the response and verifying it in the UI, as the approach here (calling an API to obtain a single volume's data) would become unfeasible when we need to validate links for a lot of resources (such as in the volume snapshots page). I suppose this would also be the ideal approach for validating most links to resources. I will make some changes in this PR to adjust this, and to include the validation in the volume snapshots page as well. -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: [email protected] For queries about this service, please contact Infrastructure at: [email protected]
