You are viewing a plain text version of this content. The canonical link for it is here.

Posted to dev@cordova.apache.org by julio cesar sanchez <jc...@gmail.com> on 2016/01/25 10:00:16 UTC

[DISCUSS] inAppBrowser plugin release

On latest release of inAppBrowser plugin the loadstart event stopped
working on android.
It has been fixed on the 1.2.1-dev.
Should we release it?

I think loadstart is very important and a lot of apps rely on it for
closing the window.

Re: [DISCUSS] inAppBrowser plugin release

Posted by julio cesar sanchez <jc...@gmail.com>.

Nobody is blaming you, this things happen, you didn't see that code and
nobody saw it was missing on the review (or nobody reviewed).

We should add tests for this.


loadstart is really important, it's used for auth or payment gateways (and
probably other uses), where you usually have to provide a "return url" when
the auth or the payment it's ok. As cordova apps are not websites and you
can't use a "return url", users redirect to a fake "ok" url, so when the
loadstart detects that it's the "ok" url it closes and the app knows that
it was ok

I'm not sure about what docs page do you talk about
I can see the events documented on
https://www.npmjs.com/package/cordova-plugin-inappbrowser, that is linked
on http://cordova.apache.org/docs/en/latest/cordova/plugins/pluginapis.html


2016-01-26 16:35 GMT+01:00 Joe Bowser <bo...@gmail.com>:

> First of all, we should agree on what InAppBrowser does.  It seems that
> people added way too much crap in it and we're stuck supporting an API that
> many maintainers are completely unaware about.  When I "broke" the API, I
> sent a pull request and asked people to review it before I merged it in.  I
> know that the Cordova list is slower than most other lists anywhere, but
> there's also the fact that most people don't know about loadStart and
> loadStop events, and they're not even explicitly in the docs (only inline
> on the GitHub docs page).
>
> Secondly, how important is this?
>
> On Tue, Jan 26, 2016 at 6:18 AM, Sergey Grebnov (Akvelon) <
> v-segreb@microsoft.com> wrote:
>
> > Looks like we don't have auto tests for InAppBrowser plugin at all. We
> > should write sanity test to ensure plugin is successfully initialized at
> > least (there is cordova.InAppBrowser.open method available). I also think
> > that it is doable to write auto tests for main events (load start/stop,
> > etc) as well.
> >
> > PS. CB-10428 has been resolved by the following PR:
> > https://github.com/apache/cordova-plugin-inappbrowser/pull/141/files?w=1
> >
> > -Sergey
> > -----Original Message-----
> > From: Nikhil Khandelwal [mailto:nikhilkh@microsoft.com]
> > Sent: Tuesday, January 26, 2016 12:43 AM
> > To: dev@cordova.apache.org
> > Subject: RE: [DISCUSS] inAppBrowser plugin release
> >
> > Looks like there is another regression w.r.t. browserify workflow
> CB-10428
> > for Cordova-inappbrowser.
> >
> > And this one for cordova-plugin-file:
> >
> https://na01.safelinks.protection.outlook.com/?url=https%3a%2f%2fissues.apache.org%2fjira%2fbrowse%2fCB-10419&data=01%7c01%7cv-segreb%40microsoft.com%7c87655d4803a54891496f08d325d090c1%7c72f988bf86f141af91ab2d7cd011db47%7c1&sdata=VT5z308fXUhduPr40kC6zA0IsPT4apTeA%2fnStCYkFG8%3d
> >
> > We might to rev those and re-release.
> >
> > -Nikhil
> >
> > -----Original Message-----
> > From: Nikhil Khandelwal [mailto:nikhilkh@microsoft.com]
> > Sent: Monday, January 25, 2016 10:23 AM
> > To: dev@cordova.apache.org
> > Subject: RE: [DISCUSS] inAppBrowser plugin release
> >
> > I'm curious what caused the regression? Why do we not have tests for this
> > important scenario?
> >
> > -Nikhil
> >
> > -----Original Message-----
> > From: Steven Gill [mailto:stevengill97@gmail.com]
> > Sent: Monday, January 25, 2016 1:19 AM
> > To: dev@cordova.apache.org
> > Subject: Re: [DISCUSS] inAppBrowser plugin release
> >
> > +1 Lets do it!
> >
> > On Mon, Jan 25, 2016 at 1:00 AM, julio cesar sanchez <
> > jcesarmobile@gmail.com
> > > wrote:
> >
> > > On latest release of inAppBrowser plugin the loadstart event stopped
> > > working on android.
> > > It has been fixed on the 1.2.1-dev.
> > > Should we release it?
> > >
> > > I think loadstart is very important and a lot of apps rely on it for
> > > closing the window.
> > >
> >  B KKKKKKKKKKKKKKKKKKKKKKKKKKKKKKKKKKKKKKKKKKKKKKKKKKKKKKKKKKKKKKKKKKKKCB
> >   [  X  ܚX K  K[XZ[    ] ][  X  ܚX P  ܙ ݘK \ X  K ܙ B  ܈ Y  ] [ۘ[    [X[
> >  K[XZ[    ] Z [    ܙ ݘK \ X  K ܙ B  B
> > KKKKKKKKKKKKKKKKKKKKKKKKKKKKKKKKKKKKKKKKKKKKKKKKKKKKKKKKKKKKKKKKKKKKCB
> > [  X  ܚX K  K[XZ[
> >    ] ][  X  ܚX P  ܙ ݘK \ X  K ܙ B  ܈ Y  ] [ۘ[    [X[     K[XZ[
> >    ] Z [    ܙ ݘK \ X  K ܙ B
> >
>