Re: be localflavor Belgium

2010-10-03 Thread Russell Keith-Magee 
2010/10/4 Łukasz Rekucki :
> On 3 October 2010 20:53, Laurent Luce  wrote:
>> Am I supposed to commit the patch? I don't think I have the privileges
>> to do that.
>
> No, only core developers can commit changes. The patch is marked RFC
> and has milestone 1.3, so it's pretty sure it won't be forgotten. I
> suspect that because Russel reviewed the patch, some other core
> developer needs to commit it (but that's just a wild guess). So sit
> back, relax and send a friendly reminder some time before feature
> freeze on October 18[1].

No - I'll commit it when I have the time. It's just that we're
focusing on major features right now, so minor feature additions are
taking a back seat.

Once October 18th and the feature freeze hits, I wlll commit -- or
another core developer will if they beat me to it.

So - Laurent - you've done everything right (and thanks for the
patch); in a couple of weeks your code should be in trunk.

Yours,
Russ Magee %-)

-- 
You received this message because you are subscribed to the Google Groups 
"Django developers" group.
To post to this group, send email to django-develop...@googlegroups.com.
To unsubscribe from this group, send email to 
django-developers+unsubscr...@googlegroups.com.
For more options, visit this group at 
http://groups.google.com/group/django-developers?hl=en.

Re: be localflavor Belgium

2010-10-03 Thread Łukasz Rekucki 
On 3 October 2010 20:53, Laurent Luce  wrote:
> Am I supposed to commit the patch? I don't think I have the privileges
> to do that.

No, only core developers can commit changes. The patch is marked RFC
and has milestone 1.3, so it's pretty sure it won't be forgotten. I
suspect that because Russel reviewed the patch, some other core
developer needs to commit it (but that's just a wild guess). So sit
back, relax and send a friendly reminder some time before feature
freeze on October 18[1].

[1]: 
http://www.djangoproject.com/weblog/2010/sep/30/django-1_3-release-schedule/

-- 
Łukasz Rekucki

-- 
You received this message because you are subscribed to the Google Groups 
"Django developers" group.
To post to this group, send email to django-develop...@googlegroups.com.
To unsubscribe from this group, send email to 
django-developers+unsubscr...@googlegroups.com.
For more options, visit this group at 
http://groups.google.com/group/django-developers?hl=en.

Re: be localflavor Belgium

2010-10-03 Thread Laurent Luce 
Am I supposed to commit the patch? I don't think I have the privileges
to do that.

Laurent

On Sep 30, 5:35 pm, Russell Keith-Magee 
wrote:
> On Fri, Oct 1, 2010 at 6:15 AM, Laurent Luce  wrote:
> > I updated the be localflavor patch to use unit tests instead of
> > doctests as requested. Can you check if this is what you want. There
> > was no unit tests in regressiontests/forms/localflavor/ so I am not
> > 100% sure I did what is expected. I took a look at the ones in
> > regressiontests/forms as models.
>
> The patch looks good, so I've just bumped the ticket to ready for checkin.
>
> Thanks for the contribution!
>
> Yours,
> Russ Magee %-)

-- 
You received this message because you are subscribed to the Google Groups 
"Django developers" group.
To post to this group, send email to django-develop...@googlegroups.com.
To unsubscribe from this group, send email to 
django-developers+unsubscr...@googlegroups.com.
For more options, visit this group at 
http://groups.google.com/group/django-developers?hl=en.

Re: be localflavor Belgium

2010-09-30 Thread Russell Keith-Magee 
On Fri, Oct 1, 2010 at 6:15 AM, Laurent Luce  wrote:
> I updated the be localflavor patch to use unit tests instead of
> doctests as requested. Can you check if this is what you want. There
> was no unit tests in regressiontests/forms/localflavor/ so I am not
> 100% sure I did what is expected. I took a look at the ones in
> regressiontests/forms as models.

The patch looks good, so I've just bumped the ticket to ready for checkin.

Thanks for the contribution!

Yours,
Russ Magee %-)

-- 
You received this message because you are subscribed to the Google Groups 
"Django developers" group.
To post to this group, send email to django-develop...@googlegroups.com.
To unsubscribe from this group, send email to 
django-developers+unsubscr...@googlegroups.com.
For more options, visit this group at 
http://groups.google.com/group/django-developers?hl=en.

Re: be localflavor Belgium

2010-09-30 Thread Laurent Luce 
I updated the be localflavor patch to use unit tests instead of
doctests as requested. Can you check if this is what you want. There
was no unit tests in regressiontests/forms/localflavor/ so I am not
100% sure I did what is expected. I took a look at the ones in
regressiontests/forms as models.

On Sep 29, 1:09 pm, Laurent Luce  wrote:
> I added a new patch to the ticket with the following changes:
> add support for 0, 04 phone number format. Wrap long
> lines.
>
> Laurent
>
> On Sep 29, 11:23 am, Gert Van Gool  wrote:
>
> > IMHO I would allow a non-spaced version. My typing my phone number as
> > 0498725202 should also work.
> > Second thing, wrap the long lines :)
>
> > Relevant ticket is #14349 (http://code.djangoproject.com/ticket/14349)
>
> > -- Gert
>
> > Mobile: +32 498725202
> > Twitter: @gvangool 
> > Web:http://gert.selentic.net
>
> > On Wed, Sep 29, 2010 at 18:26, Laurent Luce  wrote:
> > > Hello,
>
> > > I created a patch to add support for Belgium to localflavor. I
> > > attached the patch to a ticket which is marked as accepted. The patch
> > > contains the forms fields, unit tests and the doc changes.
>
> > > What do I need to do next ? Wait for someone to review the patch and
> > > commit it ?
>
> > > Laurent
>
> > > --
> > > You received this message because you are subscribed to the Google Groups
> > > "Django developers" group.
> > > To post to this group, send email to django-develop...@googlegroups.com.
> > > To unsubscribe from this group, send email to
> > > django-developers+unsubscr...@googlegroups.com
> > > .
> > > For more options, visit this group at
> > >http://groups.google.com/group/django-developers?hl=en.

-- 
You received this message because you are subscribed to the Google Groups 
"Django developers" group.
To post to this group, send email to django-develop...@googlegroups.com.
To unsubscribe from this group, send email to 
django-developers+unsubscr...@googlegroups.com.
For more options, visit this group at 
http://groups.google.com/group/django-developers?hl=en.

Re: be localflavor Belgium

2010-09-29 Thread Laurent Luce 
I added a new patch to the ticket with the following changes:
add support for 0, 04 phone number format. Wrap long
lines.

Laurent

On Sep 29, 11:23 am, Gert Van Gool  wrote:
> IMHO I would allow a non-spaced version. My typing my phone number as
> 0498725202 should also work.
> Second thing, wrap the long lines :)
>
> Relevant ticket is #14349 (http://code.djangoproject.com/ticket/14349)
>
> -- Gert
>
> Mobile: +32 498725202
> Twitter: @gvangool 
> Web:http://gert.selentic.net
>
> On Wed, Sep 29, 2010 at 18:26, Laurent Luce  wrote:
> > Hello,
>
> > I created a patch to add support for Belgium to localflavor. I
> > attached the patch to a ticket which is marked as accepted. The patch
> > contains the forms fields, unit tests and the doc changes.
>
> > What do I need to do next ? Wait for someone to review the patch and
> > commit it ?
>
> > Laurent
>
> > --
> > You received this message because you are subscribed to the Google Groups
> > "Django developers" group.
> > To post to this group, send email to django-develop...@googlegroups.com.
> > To unsubscribe from this group, send email to
> > django-developers+unsubscr...@googlegroups.com
> > .
> > For more options, visit this group at
> >http://groups.google.com/group/django-developers?hl=en.

-- 
You received this message because you are subscribed to the Google Groups 
"Django developers" group.
To post to this group, send email to django-develop...@googlegroups.com.
To unsubscribe from this group, send email to 
django-developers+unsubscr...@googlegroups.com.
For more options, visit this group at 
http://groups.google.com/group/django-developers?hl=en.

Re: be localflavor Belgium

2010-09-29 Thread Gert Van Gool 
IMHO I would allow a non-spaced version. My typing my phone number as
0498725202 should also work.
Second thing, wrap the long lines :)

Relevant ticket is #14349 (http://code.djangoproject.com/ticket/14349)

-- Gert

Mobile: +32 498725202
Twitter: @gvangool 
Web: http://gert.selentic.net



On Wed, Sep 29, 2010 at 18:26, Laurent Luce  wrote:

> Hello,
>
> I created a patch to add support for Belgium to localflavor. I
> attached the patch to a ticket which is marked as accepted. The patch
> contains the forms fields, unit tests and the doc changes.
>
> What do I need to do next ? Wait for someone to review the patch and
> commit it ?
>
> Laurent
>
> --
> You received this message because you are subscribed to the Google Groups
> "Django developers" group.
> To post to this group, send email to django-develop...@googlegroups.com.
> To unsubscribe from this group, send email to
> django-developers+unsubscr...@googlegroups.com
> .
> For more options, visit this group at
> http://groups.google.com/group/django-developers?hl=en.
>
>

-- 
You received this message because you are subscribed to the Google Groups 
"Django developers" group.
To post to this group, send email to django-develop...@googlegroups.com.
To unsubscribe from this group, send email to 
django-developers+unsubscr...@googlegroups.com.
For more options, visit this group at 
http://groups.google.com/group/django-developers?hl=en.

be localflavor Belgium

2010-09-29 Thread Laurent Luce 
Hello,

I created a patch to add support for Belgium to localflavor. I
attached the patch to a ticket which is marked as accepted. The patch
contains the forms fields, unit tests and the doc changes.

What do I need to do next ? Wait for someone to review the patch and
commit it ?

Laurent

-- 
You received this message because you are subscribed to the Google Groups 
"Django developers" group.
To post to this group, send email to django-develop...@googlegroups.com.
To unsubscribe from this group, send email to 
django-developers+unsubscr...@googlegroups.com.
For more options, visit this group at 
http://groups.google.com/group/django-developers?hl=en.