incubator-cloudstack-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From Olga Smola <olya.sm...@gmail.com>
Subject Re: UI tickets: Review request
Date Tue, 26 Jun 2012 07:34:30 GMT
Hi Pranav,

Excuse me, I can't understand what you mean from here "...however since the
backend API command hasn't been implemented for listing the specific OS
types". Could you please clarify which API should be implemented? Because
I've used already exists API commands "listOsCategories" and "listOsTypes".
For "listOsTypes" I use param "oscategoryid" to get only OsTypes from
specific Category.

Best regards,
Olga

On Mon, Jun 25, 2012 at 11:37 PM, Brian Federle <Brian.Federle@citrix.com>wrote:

> That is a good idea too, I agree.
>
> Also keep in mind that it will require a fair bit of UI testing too, since
> this would affect numerous dialog boxes and forms throughout the UI.
>
> -Brian
>
> On 6/25/12 1:04 PM, "Pranav Saxena" <pranav.saxena@citrix.com> wrote:
>
> >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.smola@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
>
>

Mime
  • Unnamed multipart/alternative (inline, None, 0 bytes)
View raw message