Hi Olga , I reviewed your code for the following bug which you raised - http://bugs.cloudstack.org/browse/CS-15292 . This looks like a more modular improvement in displaying the OS types as split categories. Your patch looks good however since the backend API command hasn't been implemented for listing the specific OS types , you'll have to probably wait a bit for this patch of yours to be merged ( Where you have written "api command" explicitly in your patch ).
The idea definitely looks good to me. @Brian/Jessica - what are your suggestions for the same ? Regards, Pranav -----Original Message----- From: Olga Smola [mailto:olya.sm...@gmail.com] Sent: Wednesday, June 20, 2012 8:56 PM To: cloudstack-dev@incubator.apache.org Subject: UI tickets: Review request Hi, Could you please review my patches for the following tickets: http://bugs.cloudstack.org/browse/CS-15026 http://bugs.cloudstack.org/browse/CS-15292 http://bugs.cloudstack.org/browse/CS-14110 http://bugs.cloudstack.org/browse/CS-15290 http://bugs.cloudstack.org/browse/CS-15286 http://bugs.cloudstack.org/browse/CS-14047 http://bugs.cloudstack.org/browse/CS-15287 - Please take a look at the ticket last comment http://bugs.cloudstack.org/browse/CS-15315 http://bugs.cloudstack.org/browse/CS-4460 - Please take a look at the latest ticket comment. There is a problem, already discussed with Jessica Wang, but not completed, still need feedback, maybe some ideas how to correct. If you have ideas, write to me and I'll forward already discussed with Jessica. Thanks, Olga
