Github user DaanHoogland commented on the pull request:
https://github.com/apache/cloudstack/pull/1134#issuecomment-160417989
Did a quick review and it looks great. One general remark; Let's keep
interface as orthogonal as possible, i.e. use 'list' as the prefix for search
functions and have a service implement it's own list method. @pdube you
probably found some guide somewhere that made you do this differently. can you
tell me where? The reason I want this is to be able to contain resource editing
in a single widget using as little services as possible.
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at [email protected] or file a JIRA ticket
with INFRA.
---
