[GitHub] [cloudstack] smlshn commented on issue #3336: Sort list of templates, serviceOfferings, diskOfferings etc in the deploy VM wizard

[GitBox <gi...@apache.org>]

smlshn commented on issue #3336: Sort list of templates, serviceOfferings, diskOfferings etc in the deploy VM wizard
URL: https://github.com/apache/cloudstack/pull/3336#issuecomment-496024077
 
 
   > This overrides the order in which we set the "sort_key" param is used by backend to return a pre-configured list of entities. That is not desirable.
   > 
   > For templates we don't support sort_key on backend yet and hence we need to sort alphabetically for the time being.
   > 
   > CC/ @rhtyd , @rafaelweingartner , @borisstoyanov
   
   Actually I think that might be more efficient however in Templates view table there is a column that "Order". Order function does not work right now but if I add sort functionality which sort every result by name, than this might be misleading. One may think it is done in purpose.
   
   Beside, the change is not only about sorting the templates in Create VM screen. diskOfferings, vmtemplates etc all need to be sorted as well.
   <img width="1316" alt="Screen Shot 2019-05-26 at 22 00 30" src="https://user-images.githubusercontent.com/8104993/58386074-53c22380-8003-11e9-9162-d3b8bed60c47.png">
   
   
   
   

----------------------------------------------------------------
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.
 
For queries about this service, please contact Infrastructure at:
users@infra.apache.org


With regards,
Apache Git Services