DaanHoogland commented on PR #11962:
URL: https://github.com/apache/cloudstack/pull/11962#issuecomment-3531896008

   > > @DaanHoogland, there is a similar use case with accounts where there are 
no UI options 
https://cloudstack.apache.org/api/apidocs-4.20/apis/linkAccountToLdap.html. 
Should this also be considered in this PR?
   > 
   > Yes @rajujith @DaanHoogland
   > 
   > The improvement issue is already present
   > 
   > #11473
   
   guys (@rajujith @kiranchavala), I don’t want to add and create a big 
beautiful PR. I’d rather implement smaller well tested changes, if you don’t 
mind. We need to have a backend change in `DomainResponse` as well to be able 
to decide whether to show the link or the unlink button. I am sure we will find 
more issues while working on this.


-- 
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.

To unsubscribe, e-mail: [email protected]

For queries about this service, please contact Infrastructure at:
[email protected]

Reply via email to