JoaoJandre commented on PR #13661: URL: https://github.com/apache/cloudstack/pull/13661#issuecomment-5047461506
> > > @abh1sar @JoaoJandre I think hiding these two columns when not using the KBOSS plugin is ok. A single suggestion: when using KBOSS, isn't it better to display `Unknown` in these columns for backups created using other providers, instead of an empty column with a gray circle? The way these are shown now may look like a bug. > > > > > > Made the change. It appears like this now: <img alt="Screenshot 2026-07-22 at 11 07 48 AM" width="1663" height="1013" src="https://private-user-images.githubusercontent.com/63767682/624906989-3b0472a0-37ad-410b-960c-8071a81e96a5.png?jwt=eyJ0eXAiOiJKV1QiLCJhbGciOiJIUzI1NiJ9.eyJpc3MiOiJnaXRodWIuY29tIiwiYXVkIjoicmF3LmdpdGh1YnVzZXJjb250ZW50LmNvbSIsImtleSI6ImtleTUiLCJleHAiOjE3ODQ3Mjc3OTEsIm5iZiI6MTc4NDcyNzQ5MSwicGF0aCI6Ii82Mzc2NzY4Mi82MjQ5MDY5ODktM2IwNDcyYTAtMzdhZC00MTBiLTk2MGMtODA3MWE4MWU5NmE1LnBuZz9YLUFtei1BbGdvcml0aG09QVdTNC1ITUFDLVNIQTI1NiZYLUFtei1DcmVkZW50aWFsPUFLSUFWQ09EWUxTQTUzUFFLNFpBJTJGMjAyNjA3MjIlMkZ1cy1lYXN0LTElMkZzMyUyRmF3czRfcmVxdWVzdCZYLUFtei1EYXRlPTIwMjYwNzIyVDEzMzgxMVomWC1BbXotRXhwaXJlcz0zMDAmWC1BbXotU2lnbmF0dXJlPWQ5YTI1YjM4OTkwNGJhNWRlZTc3ZmM2NzBmZThmNWFlODE5YzczZGVkNjM2MTRiMTIxZmUyMjA0ZDBhODRkZDkmWC1BbXotU2lnbmVkSGVhZGVycz1ob3N0JnJlc3BvbnNlLWNvbnRlbnQtdHlwZT1pbWFnZSUyRnBuZyJ9.ZUP7oyUZa6w9nzoiRvG0XI_Fpl9xOizk4OLk7Ao3q_A"> > > @abh1sar thank you. The code looks good to me. I will also do some basic testing. > > @JoaoJandre are you ok with these changes? @winterhazel I think it is fine. Although I will not have time to review or test this today, from the description and screenshots it seems ok. -- 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]
