greg-dove commented on pull request #942:
URL: https://github.com/apache/royale-asjs/pull/942#issuecomment-731850612


   As a quick follow-on to my last comment:
   If you are happy with that suggestion, there are two options - you could 
create a new pull request for that change, that does the following:
   ```
   1.
   delete the URLVariables class in MXRoyale
   2.
   remove this from the top level MXRoyaleClasses.as:
   import mx.messaging.channels.URLVariables; URLVariables;
   
   3.
   in mx.messaging.channels.DirectHTTPChannel
   add this inside the COMPILE::JS import block:
   
   import org.apache.royale.net.URLVariables;
   ```
   
   If you want to do that, I will happily accept the request this gets you 
(rightfully, given that the solution derived from your efforts) into the 
project's commit history, so your continued contributions can hopefully support 
you gaining committer rights in the future - we're definitely keen to welcome 
more active community members like yourself.
   
   The other option is that if you prefer otherwise, just let me know and I 
will make this change in the next 24 hours.
   


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


Reply via email to