Re: [PR] UI: List host OOBM details when enabled and configured [cloudstack]

[via GitHub]

Pearl1594 commented on PR #10472:
URL: https://github.com/apache/cloudstack/pull/10472#issuecomment-2688427460

   > clgtm and works in qa: 
![image](https://private-user-images.githubusercontent.com/2486961/417658180-76be879c-9710-4b0f-a2bf-869c06014773.png?jwt=eyJhbGciOiJIUzI1NiIsInR5cCI6IkpXVCJ9.eyJpc3MiOiJnaXRodWIuY29tIiwiYXVkIjoicmF3LmdpdGh1YnVzZXJjb250ZW50LmNvbSIsImtleSI6ImtleTUiLCJleHAiOjE3NDA2NzI2MDUsIm5iZiI6MTc0MDY3MjMwNSwicGF0aCI6Ii8yNDg2OTYxLzQxNzY1ODE4MC03NmJlODc5Yy05NzEwLTRiMGYtYTJiZi04NjljMDYwMTQ3NzMucG5nP1gtQW16LUFsZ29yaXRobT1BV1M0LUhNQUMtU0hBMjU2JlgtQW16LUNyZWRlbnRpYWw9QUtJQVZDT0RZTFNBNTNQUUs0WkElMkYyMDI1MDIyNyUyRnVzLWVhc3QtMSUyRnMzJTJGYXdzNF9yZXF1ZXN0JlgtQW16LURhdGU9MjAyNTAyMjdUMTYwNTA1WiZYLUFtei1FeHBpcmVzPTMwMCZYLUFtei1TaWduYXR1cmU9ZTJiMzFhNDFlZjdhYWI3NDgwYzk2ZDhjMmY1ZWZjZGVkMTc4OTZjYTg5YjAwMTM2ZGFjNzM3OGU5ZjU0MzAzYSZYLUFtei1TaWduZWRIZWFkZXJzPWhvc3QifQ.OW5mWsjLD2WOnILYQjV-17MF3pLkkHAuiyZQ35CMzoo)
   > 
   > however, I would expect the dialog to be filled with those values as well 
when re-configuring. (not sure if this would be another feature.
   
   That does make sense @DaanHoogland . This is the legacy way it behaves. I 
think it would be a enhancement, as we'd need to define a component for it.


-- 
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: commits-unsubscr...@cloudstack.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org