Mailing-List: contact callback-dev-help@incubator.apache.org; run by ezmlm
Precedence: bulk
Reply-To: callback-dev@incubator.apache.org
Received-SPF: pass (nike.apache.org: domain of simon.macdonald@gmail.com
 designates 209.85.160.47 as permitted sender)
MIME-Version: 1.0
In-Reply-To: 
 <CAM2q_UzqNYNrqW2qRV5y9KJSBN4CsC6jK82YpBOJYwm4hYLLtQ@mail.gmail.com>
References: 
 <CAM1AYQASv=7-zY0fr=mv8o6TgdmNzrzYS0g07SoLs9c=DsA8fg@mail.gmail.com>
 <CBDBCFC6.67DB%fil@adobe.com>
 <CAM1AYQA93Uy=mYwkgGx4fP8LwKfSujujk8aYy_6hF25jaaPUgg@mail.gmail.com>
 <CAM1AYQAKFoqWy5TzAkR4PyFq0a4J5tOc=qVT34vMtFZ8DWrcgg@mail.gmail.com>
 <CAM2q_UwEYLzeRnZEV8W3Weh8zDu1yhqUTaESh436A2BijkXo8A@mail.gmail.com>
 <CAE6O_-aFPEmZ_G8qc4pX_SmGsR7a1LDA8mt8ZM857MQeC3GWzQ@mail.gmail.com>
 <CAM2q_UzqNYNrqW2qRV5y9KJSBN4CsC6jK82YpBOJYwm4hYLLtQ@mail.gmail.com>
From: Simon MacDonald <simon.macdonald@gmail.com>
Date: Wed, 23 May 2012 15:52:46 -0400
Message-ID: 
 <CAM1AYQDBnQveGNdkKV7rHYtTWAW2=RHf4cBgcNncQB31+CC=Vw@mail.gmail.com>
Subject: Re: Adding HTTP status code to FileTransferError
To: callback-dev@incubator.apache.org
Content-Type: text/plain; charset=ISO-8859-1
Content-Transfer-Encoding: quoted-printable

I'll do the Android one today.

Simon Mac Donald
http://hi.im/simonmacdonald


On Wed, May 23, 2012 at 3:27 PM, Don Coleman <don.coleman@gmail.com> wrote:
> So would it make sense to re-assign the completed tickets to the
> platform maintainers?
>
> Are these the right people?
>
> iOS - Shazron
> Android - Joe
> Webworks - Michael
> Mobile Spec - Fil
>
> On Wed, May 23, 2012 at 2:46 PM, Shazron <shazron@gmail.com> wrote:
>> This shouldn't be closed until all pull requests + issue subtasks are
>> resolved. I would assume the platform maintainers will resolve the
>> sub-tasks. Once the sub-tasks are resolved, the parent issue can be clos=
ed.
>>
>> On Wed, May 23, 2012 at 11:42 AM, Don Coleman <don.coleman@gmail.com> wr=
ote:
>>
>>> I've created pull requests on github for iOS, Android and Webworks
>>>
>>> Should I close (or resolve) the JIRA ticket? =A0Or do I assign them to
>>> someone else?
>>>
>>> https://issues.apache.org/jira/browse/CB-790
>>>
>>> On Fri, May 18, 2012 at 1:12 PM, Simon MacDonald
>>> <simon.macdonald@gmail.com> wrote:
>>> > Never mind I realize now that Don sent it to the right place. Fil and
>>> > I were CC'ed on it.
>>> >
>>> > Simon Mac Donald
>>> > http://hi.im/simonmacdonald
>>> >
>>> >
>>> > On Fri, May 18, 2012 at 1:06 PM, Simon MacDonald
>>> > <simon.macdonald@gmail.com> wrote:
>>> >> It's here:
>>> >>
>>> >> https://people.apache.org/committer-index.html
>>> >>
>>> >> Don sent me the CLA and I'm going to forward it to the right place.
>>> >>
>>> >> Simon Mac Donald
>>> >> http://hi.im/simonmacdonald
>>> >>
>>> >>
>>> >> On Fri, May 18, 2012 at 1:07 PM, Filip Maj <fil@adobe.com> wrote:
>>> >>> Where do we check the list of people who've submitted the CLA?
>>> >>>
>>> >>> On 5/17/12 7:09 PM, "Simon MacDonald" <simon.macdonald@gmail.com>
>>> wrote:
>>> >>>
>>> >>>>I may have missed it but did you sign the Apache CLA? IIRC we need
>>> >>>>this before we can merge your pull requests.
>>> >>>>
>>> >>>>Simon Mac Donald
>>> >>>>http://hi.im/simonmacdonald
>>> >>>>
>>> >>>>
>>> >>>>On Thu, May 17, 2012 at 9:33 PM, Don Coleman <don.coleman@gmail.com=
>
>>> >>>>wrote:
>>> >>>>> What's the workflow for fixing issues?
>>> >>>>>
>>> >>>>> I issued pull requests on Github. =A0(Should these go to Apache
>>> instead?)
>>> >>>>>
>>> >>>>> Do I mark the JIRA issues as resolved or assign it to someone?
>>> >>>>>
>>> >>>>> android -
>>> https://github.com/apache/incubator-cordova-android/pull/22
>>> >>>>> iOS - https://github.com/apache/incubator-cordova-ios/pull/18
>>> >>>>> mobile-spec -
>>> >>>>>https://github.com/apache/incubator-cordova-mobile-spec/pull/6
>>> >>>>>
>>> >>>>> On Tue, May 15, 2012 at 9:09 PM, Shazron <shazron@gmail.com> wrot=
e:
>>> >>>>>> done
>>> >>>>>>
>>> >>>>>> On Tue, May 15, 2012 at 5:48 PM, Don Coleman <don.coleman@gmail.=
com
>>> >
>>> >>>>>>wrote:
>>> >>>>>>> Created https://issues.apache.org/jira/browse/CB-790
>>> >>>>>>>
>>> >>>>>>> Can someone assign it to me?
>>> >>>>>>>
>>> >>>>>>> On Tue, May 15, 2012 at 2:02 PM, Filip Maj <fil@adobe.com> wrot=
e:
>>> >>>>>>>> We can drop issues into the issue tracker for the missing
>>> platforms
>>> >>>>>>>>and
>>> >>>>>>>> hopefully our platform implementers will have time to get it i=
n
>>> for
>>> >>>>>>>>1.8.
>>> >>>>>>>>
>>> >>>>>>>> On 5/15/12 10:47 AM, "Don Coleman" <don.coleman@gmail.com> wro=
te:
>>> >>>>>>>>
>>> >>>>>>>>>I should be able to get most of these changes this week. =A0Af=
ter
>>> >>>>>>>>>looking the Blackberry code, I need to redo the Android patch.
>>> I'll
>>> >>>>>>>>>look at WP7 tomorrow. =A0Symbian, Bada, et al may be more
>>> challenging
>>> >>>>>>>>>since I don't have test devices.
>>> >>>>>>>>>
>>> >>>>>>>>>On Tue, May 15, 2012 at 11:56 AM, Filip Maj <fil@adobe.com>
>>> wrote:
>>> >>>>>>>>>> Sure. Thanks for taking the lead on this one Don. With your
>>> help we
>>> >>>>>>>>>>should
>>> >>>>>>>>>> be able to get this in for 1.8, possibly even for the RC
>>> >>>>>>>>>>(presumably
>>> >>>>>>>>>>later
>>> >>>>>>>>>> this week?)
>>> >>>>>>>>>>
>>> >>>>>>>>>> On 5/14/12 8:20 PM, "Don Coleman" <don.coleman@gmail.com>
>>> wrote:
>>> >>>>>>>>>>
>>> >>>>>>>>>>>Should I create an JIRA ticket for this?
>>> >>>>>>>>>>>
>>> >>>>>>>>>>>
>>> https://github.com/don/incubator-cordova-mobile-spec/tree/FileTrans
>>> >>>>>>>>>>>ferEr
>>> >>>>>>>>>>>ro
>>> >>>>>>>>>>>r
>>> >>>>>>>>>>>
>>> https://github.com/don/incubator-cordova-ios/tree/FileTransferError
>>> >>>>>>>>>>>
>>> https://github.com/don/incubator-cordova-android/tree/FileTransferE
>>> >>>>>>>>>>>rror
>>> >>>>>>>>>>>
>>> >>>>>>>>>>>I need to clean up the Android version before issuing a pull
>>> >>>>>>>>>>>request.
>>> >>>>>>>>>>>
>>> >>>>>>>>>>>I'll take a look at WP7 tomorrow.
>>> >>>>>>>>>>>
>>> >>>>>>>>>>>On Mon, May 14, 2012 at 3:00 PM, Joe Bowser <bowserj@gmail.c=
om>
>>> >>>>>>>>>>>wrote:
>>> >>>>>>>>>>>> +1
>>> >>>>>>>>>>>>
>>> >>>>>>>>>>>> On Mon, May 14, 2012 at 11:54 AM, Shazron <shazron@gmail.c=
om>
>>> >>>>>>>>>>>>wrote:
>>> >>>>>>>>>>>>
>>> >>>>>>>>>>>>> +1
>>> >>>>>>>>>>>>>
>>> >>>>>>>>>>>>> On Mon, May 14, 2012 at 11:56 AM, Filip Maj <fil@adobe.co=
m>
>>> >>>>>>>>>>>>>wrote:
>>> >>>>>>>>>>>>> > +1 !
>>> >>>>>>>>>>>>> >
>>> >>>>>>>>>>>>> > On 5/14/12 11:50 AM, "Simon MacDonald"
>>> >>>>>>>>>>>>><simon.macdonald@gmail.com>
>>> >>>>>>>>>>>>> wrote:
>>> >>>>>>>>>>>>> >
>>> >>>>>>>>>>>>> >>Sounds good to me. I'd probably skip the "http_message"=
 as
>>> you
>>> >>>>>>>>>>>>>pretty
>>> >>>>>>>>>>>>> much
>>> >>>>>>>>>>>>> >>get all you need from the status and you don't need to
>>> >>>>>>>>>>>>>localize the
>>> >>>>>>>>>>>>> >>string.
>>> >>>>>>>>>>>>> >>
>>> >>>>>>>>>>>>> >>Simon Mac Donald
>>> >>>>>>>>>>>>> >>http://hi.im/simonmacdonald
>>> >>>>>>>>>>>>> >>
>>> >>>>>>>>>>>>> >>
>>> >>>>>>>>>>>>> >>On Mon, May 14, 2012 at 9:29 AM, Don Coleman
>>> >>>>>>>>>>>>><don.coleman@gmail.com>
>>> >>>>>>>>>>>>> >>wrote:
>>> >>>>>>>>>>>>> >>
>>> >>>>>>>>>>>>> >>> I'd like to add HTTP Status Code and HTTP Status Mess=
age
>>> to
>>> >>>>>>>>>>>>> >>> FileTransferError.
>>> >>>>>>>>>>>>> >>>
>>> >>>>>>>>>>>>> >>> Any objections?
>>> >>>>>>>>>>>>> >>>
>>> >>>>>>>>>>>>> >>> {
>>> >>>>>>>>>>>>> >>> =A0 =A0 =A0 "code": FILE_NOT_FOUND_ERR
>>> >>>>>>>>>>>>> >>> =A0 =A0 =A0 "source": "http://example.com/missing.txt=
",
>>> >>>>>>>>>>>>> >>> =A0 =A0 =A0 "target": "file:///mnt/sdcard/foo.txt",
>>> >>>>>>>>>>>>> >>> =A0 =A0 =A0 "http_status": 404
>>> >>>>>>>>>>>>> >>> =A0 =A0 =A0 "http_message": "Not Found"
>>> >>>>>>>>>>>>> >>> }
>>> >>>>>>>>>>>>> >>>
>>> >>>>>>>>>>>>> >>> {
>>> >>>>>>>>>>>>> >>> =A0 =A0 =A0 "code": CONNECTION_ERR
>>> >>>>>>>>>>>>> >>> =A0 =A0 =A0 "source": "http://example.com/protected.t=
xt",
>>> >>>>>>>>>>>>> >>> =A0 =A0 =A0 "target": "file:///mnt/sdcard/bar.txt",
>>> >>>>>>>>>>>>> >>> =A0 =A0 =A0 "http_status": 401
>>> >>>>>>>>>>>>> >>> =A0 =A0 =A0 "http_message": "Authorization Required"
>>> >>>>>>>>>>>>> >>> }
>>> >>>>>>>>>>>>> >>>
>>> >>>>>>>>>>>>> >>> {
>>> >>>>>>>>>>>>> >>> =A0 =A0 =A0 "code": CONNECTION_ERR
>>> >>>>>>>>>>>>> >>> =A0 =A0 =A0 "source": "http://example.com/admin.txt",
>>> >>>>>>>>>>>>> >>> =A0 =A0 =A0 "target": "file:///mnt/sdcard/baz.txt",
>>> >>>>>>>>>>>>> >>> =A0 =A0 =A0 "http_status": 403
>>> >>>>>>>>>>>>> >>> =A0 =A0 =A0 "http_message": "Forbidden"
>>> >>>>>>>>>>>>> >>> }
>>> >>>>>>>>>>>>> >>>
>>> >>>>>>>>>>>>> >
>>> >>>>>>>>>>>>>
>>> >>>>>>>>>>
>>> >>>>>>>>
>>> >>>
>>>