knacktim commented on pull request #573: URL: https://github.com/apache/guacamole-client/pull/573#issuecomment-785192211
@mike-jumper I am going to try and work on these fixes soon. (Life is busy for me atm) > Perhaps it would be better to establish a generic, non-Angular, global object defined by the webapp that could be used as the overall Guacamole web application event target? That would allow extensions to hook into and deal with various events independently of AngularJS, and would provide the webapp with a more generic and friendly hook mechanism. I like this idea. I took a look at the Event Target that you linked above and I think I would run into the same shortcoming that I had in my use case of making a REST call in the listener function. I don't want the listener function to be marked as 'done' until the REST call out to the Identity Provider is done and I call some kind of 'resolve' or 'success' callback. I think it is feasible for me to do some refactoring on the event target to use the [Promises API](https://developer.mozilla.org/en-US/docs/Web/JavaScript/Reference/Global_Objects/Promise). Does that sound like what you had in mind? ---------------------------------------------------------------- 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: [email protected]
