Serkan,

Are you going to update that PR - that's what you mean ? If something is
not fully understandable let me know. Just add comment on GitHub.

Thanks, Piotr

2018-05-14 15:30 GMT+02:00 <serkan....@likyateknoloji.com>:

>
> Thanks Piotr,
>
> I agree with you with the comments about Wrappers to be compiled with
> JS-ONLY directives,
> not clear with the others as I am not experienced on the codebase.
>
> I am going to work on Base64Decoder while waiting for the PR confirmed.
>
>
> Serkan.
>
> Alinti Piotr Zarzycki <piotrzarzyck...@gmail.com>
>
> 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%7Ce1
>>>> e62544018643f6683b08d5b632e1a5%7Cfa7b1b5a7b34438794aed2c178d
>>>> ecee1%7C0%7C0%7C636615252079891578&sdata=C%2B1tfD48vUt6aKupj
>>>> D0fqk5vBWmxYtWCRUFnFuHh6zw%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%7Cfa7b1b5a7b34438794a
>>>> ed2c178decee1%7C0%7C0%7C636615252079901586&sdata=NxNZD3qfIJU
>>>> HbeOcWHaw5uSwa8GwetCbJ%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%7Ce1
>>>> e62544018643f6683b08d5b632e1a5%7Cfa7b1b5a7b34438794aed2c178d
>>>> ecee1%7C0%7C0%7C636615252079911594&sdata=3sGjeKNub2rLOajGVQj
>>>> 8iHWjYRWQlzZeYWnCLJ9Dwr4%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%7Cfa7b1b5a7b34438794a
>>>> ed2c178decee1%7C0%7C0%7C636615252079921602&sdata=tyYxIkfE7qs
>>>> WHf43tQ9%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>*
>>
>
>
> --
> Serkan Taş
> Mobil : +90 532 250 07 71
> Likya Bilgi Teknolojileri
> ve İletişim Hiz. Ltd. Şti.
> www.likyateknoloji.com
>
> --------------------------------------
> Bu elektronik posta ve onunla iletilen bütün dosyalar gizlidir. Sadece
> yukarıda isimleri belirtilen kişiler arasında özel haberleşme amacını
> taşımaktadır. Size yanlışlıkla ulaşmışsa bu elektonik postanın içeriğini
> açıklamanız, kopyalamanız, yönlendirmeniz ve kullanmanız kesinlikle
> yasaktır. Lütfen mesajı geri gönderiniz ve sisteminizden siliniz. Likya
> Bilgi Teknolojileri ve İletişim Hiz. Ltd. Şti. bu mesajın içeriği ile
> ilgili olarak hiç bir hukuksal sorumluluğu kabul etmez.
>
> This electronic mail and any files transmitted with it are intended for
> the private use of  the persons named above. If you received this message
> in error, forwarding, copying or use of any of the information is strictly
> prohibited. Please immediately notify the sender and delete it from your
> system. Likya Bilgi Teknolojileri ve İletişim Hiz. Ltd. Şti. does not
> accept legal responsibility for the contents of this message.
> --------------------------------------
>
>


-- 

Piotr Zarzycki

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

Reply via email to