Hi Alina, I am not seeing Tile in the list of APIs your application used. We should only be working on classes used by your app.
Thanks, -Alex On 4/4/18, 8:03 AM, "Peter Ent" <[email protected]> wrote: >Hi Alina, > >The CloseEvent looks fine. > >In the others, the [IconFile] needs to be removed. If you have removed it >I didn't see the deletion in the pull request. > >For Tile, I think that needs more work, unless I'm not reading the pull >request correctly. You should remove most of the contents of this file. >When we get to making Tile work, we have a Tile layout in Royale already >and chances the MX Tile will be implemented using the Royale TileLayout. >But if not we can always go back and bring in code from the original and >re-work it then. > >Thanks, >Peter > >On 4/4/18, 6:15 AM, "Alina Kazi" <[email protected]> wrote: > >>Branch->mxRoyaleTileandTileDirection >>https://na01.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.c >>o >>m%2Falinakazi%2Froyale-asjs%2Ftree%2FmxRoyaleTileandTileDirection&data=02 >>% >>7C01%7Cpent%40adobe.com%7C19e6271d08cd4b9abe1908d59a1502c5%7Cfa7b1b5a7b34 >>4 >>38794aed2c178decee1%7C0%7C0%7C636584337418853728&sdata=ri9QIFg3k4H9MsmYy0 >>c >>C48iy97u%2FZkSlAL0jPwlwoYE%3D&reserved=0 >>Pull Request >>:https://na01.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub. >>c >>om%2Fapache%2Froyale-asjs%2Fpull%2F143&data=02%7C01%7Cpent%40adobe.com%7C >>1 >>9e6271d08cd4b9abe1908d59a1502c5%7Cfa7b1b5a7b34438794aed2c178decee1%7C0%7C >>0 >>%7C636584337418853728&sdata=Q%2B%2FhMzYPx5Eklaicfgeo8p9S9EaAZfbMy6%2B0UXc >>v >>b5s%3D&reserved=0 >>Tile.as >>TileDirection.as >> >>Branch->MxRoyaleCloseEvent >>https://na01.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.c >>o >>m%2Falinakazi%2Froyale-asjs%2Ftree%2FMxRoyaleCloseEvent&data=02%7C01%7Cpe >>n >>t%40adobe.com%7C19e6271d08cd4b9abe1908d59a1502c5%7Cfa7b1b5a7b34438794aed2 >>c >>178decee1%7C0%7C0%7C636584337418853728&sdata=JZ4a9VfDppcJkfQr%2BKivd9wGDk >>U >>cNHO7nDxJTXFAMus%3D&reserved=0 >>Pull Request >>:https://na01.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub. >>c >>om%2Fapache%2Froyale-asjs%2Fpull%2F144&data=02%7C01%7Cpent%40adobe.com%7C >>1 >>9e6271d08cd4b9abe1908d59a1502c5%7Cfa7b1b5a7b34438794aed2c178decee1%7C0%7C >>0 >>%7C636584337418853728&sdata=V%2FcBSzGeei7i0YNTIZoEnVKJbi18DtVLv%2BV%2F2vY >>7 >>HA4%3D&reserved=0 >>CloseEvent.as >> >>Branch->mxRoyaleScrollPolicynControlBar >>https://na01.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.c >>o >>m%2Falinakazi%2Froyale-asjs%2Ftree%2FmxRoyaleScrollPolicynControlBar&data >>= >>02%7C01%7Cpent%40adobe.com%7C19e6271d08cd4b9abe1908d59a1502c5%7Cfa7b1b5a7 >>b >>34438794aed2c178decee1%7C0%7C0%7C636584337418853728&sdata=sWHbuXAg4aDMef7 >>% >>2B56eYCTvktJzqI13HmFapcTzhcn8%3D&reserved=0 >>Pull Request >>:https://na01.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub. >>c >>om%2Fapache%2Froyale-asjs%2Fpull%2F145&data=02%7C01%7Cpent%40adobe.com%7C >>1 >>9e6271d08cd4b9abe1908d59a1502c5%7Cfa7b1b5a7b34438794aed2c178decee1%7C0%7C >>0 >>%7C636584337418853728&sdata=rlggrlP1C%2BlCX07dIgA2sGoIJvA%2Frc2zleh267ZN5 >>N >>M%3D&reserved=0 >>ScrollPolicy.as >>ControlBar.as >> >>Please Review. >> >>Thanks, >>Alina Kazi >> >>-----Original Message----- >>From: Alina Kazi [mailto:[email protected]] >>Sent: Wednesday, April 04, 2018 12:06 PM >>To: [email protected] >>Subject: RE: Work on Emulation >> >>Hi Alex, >> >>Tile and Tile Direction are ready for Review. >>And Now, I am working on CloseEvent. >> >>Thanks, >>Alina >>-----Original Message----- >>From: Alex Harui [mailto:[email protected]] >>Sent: Wednesday, April 04, 2018 11:14 AM >>To: [email protected] >>Subject: Re: Work on Emulation >> >>Hi Alina, >> >>Peter meant that the changing of mx_internal to protected was only >>temporary. mx_internal should be working for Royale so please use >>mx_internal instead of protected. Using protected may cause other things >>to not work. >> >>Thanks, >>-Alex >> >>From: Alina Kazi <[email protected]> >>Reply-To: "[email protected]" <[email protected]> >>Date: Tuesday, April 3, 2018 at 10:36 PM >>To: "[email protected]" <[email protected]> >>Subject: RE: Work on Emulation >> >> >>Thanks. >>replaced mx_internal with protected for now. >>And Tile and TileDirection are compiled Successfully without any warning. >> >>Alina Kazi >> >> >>From: Peter Ent [mailto:[email protected]] >> >>Sent: Tuesday, April 03, 2018 5:33 PM >>To: [email protected] >>Subject: Re: Work on Emulation >> >> >> >>I recommended changing mx_internal to protected as a stop-gap. I knew >>Alex would be putting something out for MXRoyale. >> >> >> >>Also: please update the Github issue with your progress. I've been trying >>to be consistent using it as more people working on the same items will >>be confusing. >> >> >> >>https://na01.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.c >>o >>m%2Fapache%2Froyale-asjs%2Fissues%2F136&data=02%7C01%7Cpent%40adobe.com%7 >>C >>19e6271d08cd4b9abe1908d59a1502c5%7Cfa7b1b5a7b34438794aed2c178decee1%7C0%7 >>C >>0%7C636584337418853728&sdata=EhaUCaU3xuzqoQv0CnEpILTCx%2BeCa2qDuOD9Q%2BeP >>q >>C8%3D&reserved=0 >><https://na01.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub. >>c >>o >>m%2Fapache%2Froyale-asjs%2Fissues%2F136&data=02%7C01%7Caharui%40adobe.com >>% >>7 >>Ca8619ad543114ced36b508d599ee2969%7Cfa7b1b5a7b34438794aed2c178decee1%7C0% >>7 >>C >>0%7C636584170597362200&sdata=5Nzq5hj1BwKn8tvGMUcb8Z6FIA0hlg2KQx%2BhpsDRPd >>c >>% >>3D&reserved=0> >> >> >> >>Thanks. >> >>Peter >> >> >> >>From: >>Piotr Zarzycki <[email protected]> >>Reply-To: "[email protected]" <[email protected]> >>Date: Tuesday, April 3, 2018 at 8:26 AM >>To: "[email protected]" <[email protected]> >>Subject: Re: Work on Emulation >> >> >> >>Hi Alina, >> >> >> >>If it is mx_internal I see suggestion to change it to some >>protected/private ? I'm not sure about that, but the other way is to got >>with suggestion - create getter/setter. >> Alex also in his suppress warnings in that commits [1] >> >> >> >>[1] >>https://na01.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.c >>o >>m%2Fapache%2Froyale-asjs%2Fcommit%2F3cfa84f95635df16e5f6b8214b483a&data=0 >>2 >>%7C01%7Cpent%40adobe.com%7C19e6271d08cd4b9abe1908d59a1502c5%7Cfa7b1b5a7b3 >>4 >>438794aed2c178decee1%7C0%7C0%7C636584337418853728&sdata=6aLKAVHE6Y5iL4OAw >>8 >>mEHAZ5gtLABpDocDo5Jyskiqo%3D&reserved=0 >>e4c68a902c >><https://na01.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub. >>c >>o >>m%2Fapache%2Froyale-asjs%2Fcommit%2F3cfa84f95635df16e5f6b8214b483ae4c68a9 >>0 >>2 >>c&data=02%7C01%7Cpent%40adobe.com%7C679c7e10e41d48c27a3908d5995e21d8%7Cfa >>7 >>b >>1b5a7b34438794aed2c178decee1%7C0%7C0%7C636583551935678296&sdata=hUZODGygt >>u >>u >>b%2BmqBixRZEd0XNL0UcTlTA0h43nGoFb0%3D&reserved=0> >> >> >> >>Thanks, >> >>Piotr >> >> >> >>2018-04-03 14:22 GMT+02:00 Alina Kazi <[email protected]>: >>Thanks Alex >>Now I am working on : Tile and TileDirection >>Both files are compiled successfully after changes Pull Request : >> >>https://na01.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.c >>o >>m%2Fapache%2Froyale-asjs%2Fpull%2F143&data=02%7C01%7Cpent%40adobe.com%7C1 >>9 >>e6271d08cd4b9abe1908d59a1502c5%7Cfa7b1b5a7b34438794aed2c178decee1%7C0%7C0 >>% >>7C636584337418853728&sdata=Q%2B%2FhMzYPx5Eklaicfgeo8p9S9EaAZfbMy6%2B0UXcv >>b >>5s%3D&reserved=0 >><https://na01.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub. >>c >>o >>m%2Fapache%2Froyale-asjs%2Fpull%2F143&data=02%7C01%7Cpent%40adobe.com%7C6 >>7 >>9 >>c7e10e41d48c27a3908d5995e21d8%7Cfa7b1b5a7b34438794aed2c178decee1%7C0%7C0% >>7 >>C >>636583551935678296&sdata=lla5d0d7ftNbP7MhPqKYHjwhn8uxAjXdHLPyGvVrTHo%3D&r >>e >>s >>erved=0> >> >>Only a Warning in Tile.as >> >>What I should do to remove that warning >>[java] >>D:\outtest\royale-asjs\frameworks\projects\MXRoyale\src\main\royale\ >>mx\containers\Tile.as(277): col: 5 public var may not work in minified JS >>output . Use getter/setter instead. >> [java] >> [java] mx_internal var cellWidth:Number; >> >>Thanks, >>Alina Kazi >>From: Alex >> Harui [mailto:[email protected]] >> >>Sent: Tuesday, April 03, 2018 12:27 PM >> >>To: [email protected] >>Subject: Re: Work on Emulation >> >> >> >> >> >>Hi Alina, >> >> >> >>Congratulations! I merged your pull request. You have now contributed >>code to Apache Royale. >> >> >> >>You should be able to use mx_internal now. I pushed those changes >>earlier. >> >> >> >>I'm done for today, hopefully Peter or Piotr will review the Hrule/Vrule >>pull request. Please make sure there are no warnings in the build. I >>updated ListEvent and SliderEvent to suppress warnings. >> >> >> >>Thanks, >> >>-Alex >> >> >> >>From: >>Alina Kazi <[email protected]> >>Reply-To: "[email protected]" <[email protected]> >>Date: Monday, April 2, 2018 at 11:25 PM >>To: "[email protected]" <[email protected]> >>Subject: RE: Work on Emulation >> >> >> >>Thanks alot Peter and Alex. >>HRule and VRule Files are added to new Branch and Compiled Successfully >>https://na01.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.c >>o >>m%2Falinakazi%2Froyale-asjs%2Ftree%2Falinakazi-HRuleVRule%2Fframewor&data >>= >>02%7C01%7Cpent%40adobe.com%7C19e6271d08cd4b9abe1908d59a1502c5%7Cfa7b1b5a7 >>b >>34438794aed2c178decee1%7C0%7C0%7C636584337418853728&sdata=KqF8DsxJLOEDLeY >>D >>Q4V1Jvb4KToKq43jAGIaK%2F3UbUw%3D&reserved=0 >>ks/projects/MXRoyale/src/main/royale/mx/controls >><https://na01.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub. >>c >>o >>m%2Falinakazi%2Froyale-asjs%2Ftree%2Falinakazi-HRuleVRule%2Fframeworks%2F >>p >>r >>ojects%2FMXRoyale%2Fsrc%2Fmain%2Froyale%2Fmx%2Fcontrols&data=02%7C01%7Cah >>a >>r >>ui%40adobe.com%7Cf803a2f5bfae4b51df5508d5992bc5ff%7Cfa7b1b5a7b34438794aed >>2 >>c >>178decee1%7C0%7C0%7C636583335664948195&sdata=0wAuwW7NcLkLbUw6yQkidChhJ5t7 >>s >>x >>VR9aDt%2Fdqcbm0%3D&reserved=0> >> >>In Panel : I will also replace ³mx_internal³ with "protected" >> >>And comment-out all functions and/or variables that referencing something >>not yet ported. >> >>Alina Kazi >>From: Alex >> Harui [mailto:[email protected]] >> >>Sent: Monday, April 02, 2018 8:28 PM >>To: [email protected] >>Subject: Re: Work on Emulation >> >> >> >>Hi Alina, >> >> >> >>mx_internal methods may not need to be ported. If they are not listed in >>the list of API s you used in your app, the next question is whether >>they are needed to emulate the functionality of the component that you do >>use. In many cases they are not needed. >> >> >> >>I am working on getting mx_internal working on the JS side. Even though >>mx_internal APIs are not supported by Flex, I don't doubt that other >>namespaces are used somewhere so we should try to get it to work. It has >>been working in simple test cases so not sure yet why it isn't working in >>the emulation components. >> >> >> >>Thanks, >> >>-Alex >> >> >> >>From: >>Peter Ent <[email protected]> >>Reply-To: "[email protected]" <[email protected]> >>Date: Monday, April 2, 2018 at 7:06 AM >>To: "[email protected]" <[email protected]> >>Subject: Re: Work on Emulation >> >> >> >>Hi Alina, >> >> >> >>Sorry for the delayed response. >> >> >> >>First, try to port over something easier like VRule and HRule as a first >>step since these are less complex. >> >> >> >>As for mx_internal - I have been changing these to "protected". >> >> >> >>Once you do that you'll find that you will also need IUITextField. I >>haven't done that but I just took a look at it. You can do a couple of >>things: >> >> >> >>In Panel (and any where else), comment-out any function or variable that >>references something not yet ported. We are trying to pass compilation, >>so that would be OK at this stage. >> >> >> >>If you feel you need to bring something else over, then you have to go >>through the same process of course. Since you can wind up following >>class and interface down a long chain, I would just comment-out things >>complex. Sometimes I am left with "implements IUIComponent" and the rest >>of the interfaces listed in a class are put into comments. >> >> >> >>I hope that helps. >> >>‹peter >> >> >> >>From: >>Alina Kazi <[email protected]> >>Reply-To: "[email protected]" <[email protected]> >>Date: Monday, April 2, 2018 at 7:43 AM >>To: "[email protected]" <[email protected]> >>Subject: RE: Work on Emulation >> >> >> >>mx.containers.Panel >> >>Panel.as file contains many variables and methods that are using >>mx_internal e.g. Variable >> mx_internal var _showCloseButton:Boolean = false; >> >>Method >> mx_internal function getStatusTextField():IUITextField >> { >> return statusTextField; >> } >>As use namespace mx_internal; and import mx.core.mx_internal; are >>commented in source files . >>I am getting errors on these variables and methods e.g. >> [java] >>D:\out\royale-asjs\frameworks\projects\MXRoyale\src\main\royale\mx\contai >>n >>e >>rs\Panel.as(1796): >> >>col: 6 function 'getStatusTextField' will be scoped to the default >>namespace: mx.containers.Panel: >> internal. >>It will not be visible outside of this package. >> [java] >> [java] function getStatusTextField():IUITextField >> >> >>Should I comment all of these variables and methods ? >> >>Or >>Should I make all of these variables and methods Public? >> >>For Reference, I am sharing link. So you don't need to find this file in >>flex-sdk , You can find the at: >>Flex-sdk mx.containers.Panel: >> >>https://na01.safelinks.protection.outlook.com/?url=https%3A%2F%2Fwww.drop >>b >>ox.com%2Fs%2F6xb1jp2efwuwnam%2FPanelFlex.as%3Fdl%3D0&data=02%7C01%7Cpent% >>4 >>0adobe.com%7C19e6271d08cd4b9abe1908d59a1502c5%7Cfa7b1b5a7b34438794aed2c17 >>8 >>decee1%7C0%7C0%7C636584337418853728&sdata=gu%2BhySrmf%2FUKFngpLQzeWcROCfK >>e >>%2FxzorPgxOHjwqis%3D&reserved=0 >><https://na01.safelinks.protection.outlook.com/?url=https%3A%2F%2Fwww.dro >>p >>b >>ox.com%2Fs%2F6xb1jp2efwuwnam%2FPanelFlex.as%3Fdl%3D0&data=02%7C01%7Cpent% >>4 >>0 >>adobe.com%7C81660be1d8504effc16c08d5988f0c53%7Cfa7b1b5a7b34438794aed2c178 >>d >>e >>cee1%7C0%7C0%7C636582662509500604&sdata=xLjL8AM45ZA2E8IRILbskD04cJCL9SVSc >>5 >>F >>i0%2BO98Tg%3D&reserved=0> >>Royale >>:https://na01.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub. >>c >>om%2Falinakazi%2Froyale-asjs%2Fblob%2FmxRoyalePanel%2Fframeworks%2Fpro&da >>t >>a=02%7C01%7Cpent%40adobe.com%7C19e6271d08cd4b9abe1908d59a1502c5%7Cfa7b1b5 >>a >>7b34438794aed2c178decee1%7C0%7C0%7C636584337418863731&sdata=a8xVdkw2mnwSo >>j >>xLoUJiZmOIN0SxdeqviB%2By5AbaSl4%3D&reserved=0 >>jects/MXRoyale/src/main/royale/mx/containers/Panel.as >><https://na01.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub. >>c >>o >>m%2Falinakazi%2Froyale-asjs%2Fblob%2FmxRoyalePanel%2Fframeworks%2Fproject >>s >>% >>2FMXRoyale%2Fsrc%2Fmain%2Froyale%2Fmx%2Fcontainers%2FPanel.as&data=02%7C0 >>1 >>% >>7Cpent%40adobe.com%7C81660be1d8504effc16c08d5988f0c53%7Cfa7b1b5a7b3443879 >>4 >>a >>ed2c178decee1%7C0%7C0%7C636582662509500604&sdata=Zwbl%2B2eMeQIsuoDCI4mfZ1 >>A >>4 >>TF3cA%2FcSqlzq53DEO5Y%3D&reserved=0> >> >>Thanks in Advance. >>Alina >> >>From: Alina >> Kazi [mailto:[email protected]] >> >>Sent: Monday, April 02, 2018 3:25 PM >>To: [email protected] >>Subject: RE: Work on Emulation >> >> >> >> >>Thanks >>J >>New branch Created : >> >>https://na01.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.c >>o >>m%2Falinakazi%2Froyale-asjs%2Ftree%2FmxRoyalePanel&data=02%7C01%7Cpent%40 >>a >>dobe.com%7C19e6271d08cd4b9abe1908d59a1502c5%7Cfa7b1b5a7b34438794aed2c178d >>e >>cee1%7C0%7C0%7C636584337418863731&sdata=e43MWXidxAVYCz6uJEwPWPigWnXnlKpgh >>0 >>012hW2VmA%3D&reserved=0 >><https://na01.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub. >>c >>o >>m%2Falinakazi%2Froyale-asjs%2Ftree%2FmxRoyalePanel&data=02%7C01%7Cpent%40 >>a >>d >>obe.com%7C81660be1d8504effc16c08d5988f0c53%7Cfa7b1b5a7b34438794aed2c178de >>c >>e >>e1%7C0%7C0%7C636582662509500604&sdata=AbVDtIurdRp7fwqDGrLxUrQqYXv8EvuTwCD >>w >>7 >>%2BWMBtc%3D&reserved=0> >> >>Copied MX Panel from flex-sdk to New branch (mxRoyalePanel) Pull Request >>: >> >>https://na01.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.c >>o >>m%2Fapache%2Froyale-asjs%2Fpull%2F141&data=02%7C01%7Cpent%40adobe.com%7C1 >>9 >>e6271d08cd4b9abe1908d59a1502c5%7Cfa7b1b5a7b34438794aed2c178decee1%7C0%7C0 >>% >>7C636584337418863731&sdata=0%2Br4jSVY%2B3WoupxT%2BxU8noG6M3ZuN2labOIcdB%2 >>F >>t9%2FY%3D&reserved=0 >><https://na01.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub. >>c >>o >>m%2Fapache%2Froyale-asjs%2Fpull%2F141&data=02%7C01%7Cpent%40adobe.com%7C8 >>1 >>6 >>60be1d8504effc16c08d5988f0c53%7Cfa7b1b5a7b34438794aed2c178decee1%7C0%7C0% >>7 >>C >>636582662509500604&sdata=sVStAHTosnX0lBpTYWRCEC23BiiVaX8t1%2F08YICYjOs%3D >>& >>r >>eserved=0> >> >>-Alina Kazi >> >> >>From: Piotr >> Zarzycki [mailto:[email protected]] >> >>Sent: Monday, April 02, 2018 1:22 PM >>To: [email protected] >>Subject: Re: Work on Emulation >> >>Alina, >> >> >>You cannot delete pull request. You can only close it, so we are good >>with >>139 and 140. They are closed. >> >> >> >>Your pull request 138 now looks great! :) >> >> >> >>Whenever you are going to push some bigger changes you should create new >>branches. If you are working right now on Panel - probably new branch >>with your changes would be the best. Our pull request will looks than >>cleaner. - That is just my recommendation. >> >> >> >>In general congrats for your first pull request to Apache Royale >>project!! >>:) >> >> >> >>Thanks, >> >>Piotr >> >> >> >>2018-04-02 8:08 GMT+02:00 Alina Kazi <[email protected]>: >>Hi Piotr, >> >>Thanks Piotr. >> >>I am trying to add another pull request for SliderEvent.as file. but when >>I added this file manually at >>https://na01.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.c >>o >>m%2Falinakazi%2Froyale-asjs%2Ftree%2FmxRoyaleAQ%2Fframeworks%2Fproject&da >>t >>a=02%7C01%7Cpent%40adobe.com%7C19e6271d08cd4b9abe1908d59a1502c5%7Cfa7b1b5 >>a >>7b34438794aed2c178decee1%7C0%7C0%7C636584337418863731&sdata=nQ1Yt%2FwJxyU >>U >>ybkTdCMHxgoxZ%2BaqMXv9gOJFXqo1agQ%3D&reserved=0 >>s >>/MXRoyale/src/main/royale/mx/events >><https://na01.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub. >>c >>o >>m%2Falinakazi%2Froyale-asjs%2Ftree%2FmxRoyaleAQ%2Fframeworks%2Fprojects%2 >>5 >>0 >>d%250a%2FMXRoyale%2Fsrc%2Fmain%2Froyale%2Fmx%2Fevents&data=02%7C01%7Cpent >>% >>4 >>0adobe.com%7C81660be1d8504effc16c08d5988f0c53%7Cfa7b1b5a7b34438794aed2c17 >>8 >>d >>ecee1%7C0%7C0%7C636582662509500604&sdata=rhi6Z0g%2B6UCXr%2Btgp5gpKFrmdaaj >>q >>Y >>LlmTh7iNvdoCU%3D&reserved=0> >>and commit it. >>Commit auto added to previous Pull Request : >>https://na01.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.c >>o >>m%2Fapache%2Froyale-asjs%2Fpull%2F138&data=02%7C01%7Cpent%40adobe.com%7C1 >>9 >>e6271d08cd4b9abe1908d59a1502c5%7Cfa7b1b5a7b34438794aed2c178decee1%7C0%7C0 >>% >>7C636584337418863731&sdata=pqviRdd4rBRgwqPalenRjO5HljItXWOLVQXHtbvvle0%3D >>& >>reserved=0 >><https://na01.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub. >>c >>o >>m%2Fapache%2Froyale-asjs%2Fpull%2F138&data=02%7C01%7Cpent%40adobe.com%7C8 >>1 >>6 >>60be1d8504effc16c08d5988f0c53%7Cfa7b1b5a7b34438794aed2c178decee1%7C0%7C0% >>7 >>C >>636582662509500604&sdata=a1%2BNp%2Ft3M7aSPIwqN%2BQt7XY50OX%2BSwRDB%2FACRJ >>h >>W >>LH4%3D&reserved=0> >> >>Is it ok to do multiple commits on same pull Request instead of creating >>New Pull Request for each file? >> >>How Can I delete these Closed Pull Requests? >>https://na01.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.c >>o >>m%2Fapache%2Froyale-asjs%2Fpull%2F139&data=02%7C01%7Cpent%40adobe.com%7C1 >>9 >>e6271d08cd4b9abe1908d59a1502c5%7Cfa7b1b5a7b34438794aed2c178decee1%7C0%7C0 >>% >>7C636584337418863731&sdata=O%2BQydj0NU08TgdGr8jlB%2B5XAWpOm43BMhggA6O7OlL >>Q >>%3D&reserved=0 >><https://na01.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub. >>c >>o >>m%2Fapache%2Froyale-asjs%2Fpull%2F139&data=02%7C01%7Cpent%40adobe.com%7C8 >>1 >>6 >>60be1d8504effc16c08d5988f0c53%7Cfa7b1b5a7b34438794aed2c178decee1%7C0%7C0% >>7 >>C >>636582662509500604&sdata=apy1DAJqmHGIjCdQDz4g0K36NbZATtHhA%2Foqc659Bjs%3D >>& >>r >>eserved=0> >>and >>https://na01.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.c >>o >>m%2Fapache%2Froyale-asjs%2Fpull%2F140&data=02%7C01%7Cpent%40adobe.com%7C1 >>9 >>e6271d08cd4b9abe1908d59a1502c5%7Cfa7b1b5a7b34438794aed2c178decee1%7C0%7C0 >>% >>7C636584337418863731&sdata=6eljm%2FuiMELY9NLSm3lt9mV3%2BfInbFfsAWuo4ZZUMS >>M >>%3D&reserved=0 >><https://na01.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub. >>c >>o >>m%2Fapache%2Froyale-asjs%2Fpull%2F140&data=02%7C01%7Cpent%40adobe.com%7C8 >>1 >>6 >>60be1d8504effc16c08d5988f0c53%7Cfa7b1b5a7b34438794aed2c178decee1%7C0%7C0% >>7 >>C >>636582662509500604&sdata=XehX29ItsGAwo33xue7KI9uD4PEJS3gGhS7ad1E7iSk%3D&r >>e >>s >>erved=0> >> >> >>-Alina >> >> >> >>-----Original Message----- >>From: piotrz [mailto:[email protected]] >>Sent: Friday, March 30, 2018 5:14 PM >>To: >>[email protected] <mailto:[email protected]> >>Subject: Re: Work on Emulation >>I see your new pull request and it doesn't looks good. [1] As you can see >>on the screenshot you did it against develop from feature/MXRoyale and it >>contains 94 commits. >> >><https://na01.safelinks.protection.outlook.com/?url=http%3A%2F%2Fapache-r >>o >>yale-users.20374.n8.nabble.com%2Ffile%2Ft1%2Fnew_pull_request.pn&data=02% >>7 >>C01%7Cpent%40adobe.com%7C19e6271d08cd4b9abe1908d59a1502c5%7Cfa7b1b5a7b344 >>3 >>8794aed2c178decee1%7C0%7C0%7C636584337418863731&sdata=rHNZfTbTa2Z8iLXtslP >>o >>gffyHAy%2B76ZnN7AgZytmsSw%3D&reserved=0 >>g >><https://na01.safelinks.protection.outlook.com/?url=http%3A%2F%2Fapache-r >>o >>y >>ale-users.20374.n8.nabble.com%2Ffile%2Ft1%2Fnew_pull_request.png&data=02% >>7 >>C >>01%7Cpent%40adobe.com%7C81660be1d8504effc16c08d5988f0c53%7Cfa7b1b5a7b3443 >>8 >>7 >>94aed2c178decee1%7C0%7C0%7C636582662509500604&sdata=O0UU8UF57g%2BxBjEm1BU >>% >>2 >>BDdoyAqaNEMxwl%2FQijXxEDKQ%3D&reserved=0> >>> >> >>The old one is being done correctly. You have used your branch -> >>apache:feature/MXRoyale from alinakazi:mxRoyaleAQ >> >><https://na01.safelinks.protection.outlook.com/?url=http%3A%2F%2Fapache-r >>o >>yale-users.20374.n8.nabble.com%2Ffile%2Ft1%2Fold_pull_Request.pn&data=02% >>7 >>C01%7Cpent%40adobe.com%7C19e6271d08cd4b9abe1908d59a1502c5%7Cfa7b1b5a7b344 >>3 >>8794aed2c178decee1%7C0%7C0%7C636584337418863731&sdata=Q1RLtZlGQgzJ3ZIGY3I >>P >>zf168%2BcwK2RY5I4agLcLnK0%3D&reserved=0 >>g >><https://na01.safelinks.protection.outlook.com/?url=http%3A%2F%2Fapache-r >>o >>y >>ale-users.20374.n8.nabble.com%2Ffile%2Ft1%2Fold_pull_Request.png&data=02% >>7 >>C >>01%7Cpent%40adobe.com%7C81660be1d8504effc16c08d5988f0c53%7Cfa7b1b5a7b3443 >>8 >>7 >>94aed2c178decee1%7C0%7C0%7C636582662509500604&sdata=wY1dxwYu%2B2G%2Be3czh >>r >>a >>yFYw7FGgBmQIqic7v26mzqEk%3D&reserved=0> >>> >> >>[1] >>https://na01.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.c >>o >>m%2Fapache%2Froyale-asjs%2Fpull%2F139%2Fcommits&data=02%7C01%7Cpent%40ado >>b >>e.com%7C19e6271d08cd4b9abe1908d59a1502c5%7Cfa7b1b5a7b34438794aed2c178dece >>e >>1%7C0%7C0%7C636584337418863731&sdata=T002sFfbImGVBpkSiCiHDrfobQS9vAhgEnZ4 >>i >>jOJq9A%3D&reserved=0 >><https://na01.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub. >>c >>o >>m%2Fapache%2Froyale-asjs%2Fpull%2F139%2Fcommits&data=02%7C01%7Cpent%40ado >>b >>e >>.com%7C81660be1d8504effc16c08d5988f0c53%7Cfa7b1b5a7b34438794aed2c178decee >>1 >>% >>7C0%7C0%7C636582662509500604&sdata=HUrVPryVBIrheFbS%2BzRap%2BPpOrVETRJB%2 >>F >>h >>630nCVWPo%3D&reserved=0> >> >>Thanks, >>Piotr >> >> >> >>-- >>Sent from: >>https://na01.safelinks.protection.outlook.com/?url=http%3A%2F%2Fapache-ro >>y >>ale-users.20374.n8.nabble.com%2F&data=02%7C01%7Cpent%40adobe.com%7C19e627 >>1 >>d08cd4b9abe1908d59a1502c5%7Cfa7b1b5a7b34438794aed2c178decee1%7C0%7C0%7C63 >>6 >>584337418863731&sdata=HZlkRMQseCJzWNErp9aIFkPVDXSfLN1wJgcwLdil%2FT4%3D&re >>s >>erved=0 >><https://na01.safelinks.protection.outlook.com/?url=http%3A%2F%2Fapache-r >>o >>y >>ale-users.20374.n8.nabble.com%2F&data=02%7C01%7Cpent%40adobe.com%7C81660b >>e >>1 >>d8504effc16c08d5988f0c53%7Cfa7b1b5a7b34438794aed2c178decee1%7C0%7C0%7C636 >>5 >>8 >>2662509500604&sdata=pfDAW81u489DuWy0Gc80ll1HzseZk5gMhoneBuYpCJQ%3D&reserv >>e >>d >>=0> >> >> >> >> >> >> >> >> >>-- >> >>Piotr Zarzycki >>Patreon: >>https://na01.safelinks.protection.outlook.com/?url=https%3A%2F%2Fwww.patr >>e >>on.com%2Fpiotrzarzycki&data=02%7C01%7Cpent%40adobe.com%7C19e6271d08cd4b9a >>b >>e1908d59a1502c5%7Cfa7b1b5a7b34438794aed2c178decee1%7C0%7C0%7C636584337418 >>8 >>63731&sdata=%2BKhmm2BcQez99eBDaV5OOy%2BzAbRWku6eP318jSrVAAU%3D&reserved=0 >><https://na01.safelinks.protection.outlook.com/?url=https%3A%2F%2Fwww.pat >>r >>e >>on.com%2Fpiotrzarzycki&data=02%7C01%7Cpent%40adobe.com%7C81660be1d8504eff >>c >>1 >>6c08d5988f0c53%7Cfa7b1b5a7b34438794aed2c178decee1%7C0%7C0%7C6365826625095 >>0 >>0 >>604&sdata=7xFAD19Wk39S%2BJEn6QB6jibV8%2B7Y9O8Ee16CAVyZPTk%3D&reserved=0> >> >> >> >> >> >> >> >> >> >> >> >> >> >> >> >> >> >> >> >> >> >> >> >> >> >> >> >>-- >> >>Piotr Zarzycki >>Patreon: >>https://na01.safelinks.protection.outlook.com/?url=https%3A%2F%2Fwww.patr >>e >>on.com%2Fpiotrzarzycki&data=02%7C01%7Cpent%40adobe.com%7C19e6271d08cd4b9a >>b >>e1908d59a1502c5%7Cfa7b1b5a7b34438794aed2c178decee1%7C0%7C0%7C636584337418 >>8 >>63731&sdata=%2BKhmm2BcQez99eBDaV5OOy%2BzAbRWku6eP318jSrVAAU%3D&reserved=0 >><https://na01.safelinks.protection.outlook.com/?url=https%3A%2F%2Fwww.pat >>r >>e >>on.com%2Fpiotrzarzycki&data=02%7C01%7Cpent%40adobe.com%7C679c7e10e41d48c2 >>7 >>a >>3908d5995e21d8%7Cfa7b1b5a7b34438794aed2c178decee1%7C0%7C0%7C6365835519356 >>7 >>8 >>296&sdata=ouHh9HJGSsN0f2%2B7vw5GDz2bcplJFMmO1DDodVy9%2BNo%3D&reserved=0> >> >> >> >
