Thank you Piotr,

They are not ready. I will inform you when i prepare PR update.

Serkan.

Alinti Piotr Zarzycki <[email protected]>

Serkan,

If you have your changes ready after my comments, please update your pull
request and let me know!

Piotr

2018-05-16 9:31 GMT+02:00 <[email protected]>:


Hi Piotr,

Rather than asking on the list, i would like to check first if you had
time to see my coments that i made few days ago.

Thanks


Serkan.

Alinti Piotr Zarzycki <[email protected]>

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 <[email protected]>:


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 <[email protected]>

Serkan,


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

Piotr

2018-05-13 23:14 GMT+02:00 Serkan Taş <[email protected]>:

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ş <[email protected]>:

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ş <[email protected]>
<[email protected]>
*Reply-To: *"[email protected]" <[email protected]>
<[email protected]> <[email protected]>
*Date: *Wednesday, May 9, 2018 at 10:00 PM
*To: *"[email protected]" <[email protected]>
<[email protected]> <[email protected]>
*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ş" <[email protected]> <
[email protected]> 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>*



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


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

Reply via email to