[GitHub] pbankonier commented on issue #3029: project - infrastructure - "Select View: Group by Account" (ui problem)

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

pbankonier commented on issue #3029: project - infrastructure - "Select View: Group by Account" (ui problem)
URL: https://github.com/apache/cloudstack/issues/3029#issuecomment-446606386
 
 
   First of all ignore my previous referenced commit, it was a fail.
   
   I analyzed this and the problem in the ui is that there is an error dialog displayed for every failed request and since it fails for every account there are a lot. Maybe there should only be one error dialog displayed for every error type.
   But I think the more significant problem here is that this will always fail, because when you are in a project view and you group the virtual routers by account your are sending the account and the project with the request, which is not valid regarding the backend.
   There are no virtual routers to group by accounts for a project because the routers created in a project are linked to the project and not to a user account anymore and all routers for the project are already displayed in the normal list.
   So I think the best solution for this behavior is to remove the group by account option in the project view.
   Furthermore it might be useful to include a possibility to show the routers linked to a project on the domain level in the group by account list (there are views where in the account column the project name is displayed) or make a new group by project list.
   
   @rhtyd What do you think of this solution?

----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on 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