Serkan,

I'm sorry about that. You should now see the comments.

Piotr

2018-05-13 23:14 GMT+02:00 Serkan Taş <serkan....@likyateknoloji.com>:

> Thanks Piotr,
>
> Can you please share your comments with me also, cause i can not see them
> on github (may be i am missing something)
>
> I am looking forward.
> Serkan
>
> 13.05.2018 13:57 tarihinde Piotr Zarzycki yazdı:
>
> Hi Serkan,
>
> I just finished review of your pull request. I have added bunch of
> comments. It would be great to have some insight from other PMCs, cause I'm
> not sure about some things.
>
> Thanks,
> Piotr
>
> 2018-05-10 20:46 GMT+02:00 Serkan Taş <serkan....@likyateknoloji.com>:
>
>> Sure that it is working with the test application but i'll give a try to
>> have wrappers and than create PR.
>>
>> 10.05.2018 10:29 tarihinde Alex Harui yazdı:
>>
>> Hi Serkan,
>>
>>
>>
>> Does that mean you have a working mx.utils.Base64Encoder?  If so, that’s
>> great news!  Please file a Pull Request with your code.
>>
>>
>>
>> Thanks,
>>
>> -Alex
>>
>>
>>
>> *From: *Serkan Taş <serkan....@likyateknoloji.com>
>> <serkan....@likyateknoloji.com>
>> *Reply-To: *"users@royale.apache.org" <users@royale.apache.org>
>> <users@royale.apache.org> <users@royale.apache.org>
>> *Date: *Wednesday, May 9, 2018 at 10:00 PM
>> *To: *"users@royale.apache.org" <users@royale.apache.org>
>> <users@royale.apache.org> <users@royale.apache.org>
>> *Subject: *Re: <inject_html> is being broken by the compiler
>>
>>
>>
>> Hi Piotr and Alex,
>>
>> Piotr;
>>
>> I worked on your code and it was starting point of my implementation.
>> Tank you so much.
>>
>> Alex,
>>
>> I now see the that i missed meaning of  "has to go in the ASDoc for the
>> constructor"   in your mail belongs to previous thread.
>>
>> Sorry for misunderstanding and thank you for your clarification.
>>
>> To show the point mixed up my mind for your referance :
>>
>> NOT Working Code :
>>
>> package mx.utils
>> {
>>
>>     public class BaseTester
>>     {
>>
>>         /**
>>          * The Royale Compiler will inject html into the index.html
>> file.  Surround with
>>          * "inject_html" tag as follows:
>>          *
>>          * <inject_html>
>>          * <script type="text/javascript" src="http://rawgit.com/beatgam
>> mit/base64-js/master/base64js.min.js"
>> <https://na01.safelinks.protection.outlook.com/?url=http%3A%2F%2Frawgit.com%2Fbeatgammit%2Fbase64-js%2Fmaster%2Fbase64js.min.js&data=02%7C01%7Caharui%40adobe.com%7Ce1e62544018643f6683b08d5b632e1a5%7Cfa7b1b5a7b34438794aed2c178decee1%7C0%7C0%7C636615252079891578&sdata=C%2B1tfD48vUt6aKupjD0fqk5vBWmxYtWCRUFnFuHh6zw%3D&reserved=0>
>> ></script>
>>          * <script type="text/javascript" src="http://rawgit.com/coolaj8
>> 6/TextEncoderLite/master/text-encoder-lite.min.js"
>> <https://na01.safelinks.protection.outlook.com/?url=http%3A%2F%2Frawgit.com%2Fcoolaj86%2FTextEncoderLite%2Fmaster%2Ftext-encoder-lite.min.js&data=02%7C01%7Caharui%40adobe.com%7Ce1e62544018643f6683b08d5b632e1a5%7Cfa7b1b5a7b34438794aed2c178decee1%7C0%7C0%7C636615252079901586&sdata=NxNZD3qfIJUHbeOcWHaw5uSwa8GwetCbJ%2FnqOm6jOkw%3D&reserved=0>
>> ></script>
>>          * </inject_html>
>>          */
>>         */** */*
>>         public function BaseTester()
>>         {
>>         }
>>     }
>> }
>>
>> Working Code :
>>
>> package mx.utils
>> {
>>
>>     public class BaseTester
>>     {
>>
>>         /**
>>          * The Royale Compiler will inject html into the index.html
>> file.  Surround with
>>          * "inject_html" tag as follows:
>>          *
>>          * <inject_html>
>>          * <script type="text/javascript" src="http://rawgit.com/beatgam
>> mit/base64-js/master/base64js.min.js"
>> <https://na01.safelinks.protection.outlook.com/?url=http%3A%2F%2Frawgit.com%2Fbeatgammit%2Fbase64-js%2Fmaster%2Fbase64js.min.js&data=02%7C01%7Caharui%40adobe.com%7Ce1e62544018643f6683b08d5b632e1a5%7Cfa7b1b5a7b34438794aed2c178decee1%7C0%7C0%7C636615252079911594&sdata=3sGjeKNub2rLOajGVQj8iHWjYRWQlzZeYWnCLJ9Dwr4%3D&reserved=0>
>> ></script>
>>          * <script type="text/javascript" src="http://rawgit.com/coolaj8
>> 6/TextEncoderLite/master/text-encoder-lite.min.js"
>> <https://na01.safelinks.protection.outlook.com/?url=http%3A%2F%2Frawgit.com%2Fcoolaj86%2FTextEncoderLite%2Fmaster%2Ftext-encoder-lite.min.js&data=02%7C01%7Caharui%40adobe.com%7Ce1e62544018643f6683b08d5b632e1a5%7Cfa7b1b5a7b34438794aed2c178decee1%7C0%7C0%7C636615252079921602&sdata=tyYxIkfE7qsWHf43tQ9%2BlI7RkGULPGyWhPCxEqiP2hk%3D&reserved=0>
>> ></script>
>>          * </inject_html>
>>          */
>>         public function BaseTester()
>>         {
>>         }
>>     }
>> }
>>
>>
>>
>> 10.05.2018 01:04 tarihinde Alex Harui yazdı:
>>
>> Hi Serkan,
>>
>>
>>
>> It is hard to tell from your description what you are trying, but as I 
>> mentioned in the other thread, the <inject_html> must be in an ASDoc comment 
>> for the constructor function of a class.  You can't put it anywhere in the 
>> class file.
>>
>>
>>
>> Show us the code that isn't working if you are still not understanding what 
>> I am saying.
>>
>>
>>
>> Thanks,
>>
>> -Alex
>>
>>
>>
>> On 5/9/18, 1:45 PM, "Serkan Taş" <serkan....@likyateknoloji.com> 
>> <serkan....@likyateknoloji.com> wrote:
>>
>>
>>
>>     Hi Alex,
>>
>>
>>
>>     After working on the probabilities, i discovered that under several
>>
>>     circumstances the <inject_html> is being broken by the compiler.
>>
>>
>>
>>     Here is my case :
>>
>>
>>
>>     1. I created a simple class
>>
>>
>>
>>     2. Inserted inject_html section as first item in class
>>
>>
>>
>>     3. Built the project and can see that the script are included.
>>
>>
>>
>>     4. Put any definition and/or declaration between the inject_html and
>>
>>     constructor - even empty comment like /** */
>>
>>
>>
>>     5. The compiler fails to generate the script includes in the index.html
>>
>>
>>
>>     This may be bug or something ?
>>
>>
>>
>>
>>
>>
>>
>>
>>
>>
>>
>
>
> --
>
> Piotr Zarzycki
>
> Patreon: *https://www.patreon.com/piotrzarzycki
> <https://www.patreon.com/piotrzarzycki>*
>
>
>


-- 

Piotr Zarzycki

Patreon: *https://www.patreon.com/piotrzarzycki
<https://www.patreon.com/piotrzarzycki>*

Reply via email to