incubator-cloudstack-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "mice xia" <mice_...@tcloudcomputing.com>
Subject Re: Review Request: add ui for resize volume
Date Tue, 05 Feb 2013 02:39:37 GMT

-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/9298/#review16079
-----------------------------------------------------------


is it better if we list diskoffering with a dropdown list, and show the disk size input field
when users choose 'custom disk offering', just like the 'create volume' dialog?

- mice xia


On Feb. 5, 2013, 2:31 a.m., Hongtu Zang wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/9298/
> -----------------------------------------------------------
> 
> (Updated Feb. 5, 2013, 2:31 a.m.)
> 
> 
> Review request for cloudstack, mice xia and Marcus Sorensen.
> 
> 
> Description
> -------
> 
> add ui for resizeVolume.
> use updateResourceCount as icon of resizeVolume.
> I don't know how to translate the label, so I use English in all files.
> 
> 
> This addresses bug CLOUDSTACK-644.
> 
> 
> Diffs
> -----
> 
>   client/WEB-INF/classes/resources/messages.properties 390738f 
>   client/WEB-INF/classes/resources/messages_fr_FR.properties 4ba3c72 
>   client/WEB-INF/classes/resources/messages_ja.properties 7555b83 
>   client/WEB-INF/classes/resources/messages_pt_BR.properties 3bc435b 
>   client/WEB-INF/classes/resources/messages_ru_RU.properties dd7e746 
>   client/WEB-INF/classes/resources/messages_zh_CN.properties 62160d9 
>   ui/css/cloudstack3.css eb42843 
>   ui/dictionary.jsp 203c9da 
>   ui/scripts/storage.js dd81378 
> 
> Diff: https://reviews.apache.org/r/9298/diff/
> 
> 
> Testing
> -------
> 
> Input newSize and newOfferingId.
> The api will be sent and returns correct.
> NewSize should be GB and newOfferingId find in database, not the uuid.
> 
> 
> Thanks,
> 
> Hongtu Zang
> 
>


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