Context Navigation

← Previous Ticket
Next Ticket →

#1151 new defect

connect to gabble immediately after registration

Reported by:	dsd	Owned by:	tomeu
Priority:	Low	Milestone:	Unspecified
Component:	Sugar	Version:	Unspecified
Severity:	Major	Keywords:	patch
Cc:	sascha_silbe, martin.langhoff, rgs, bernie	Distribution/OS:	Unspecified
Bug Status:	New

Description

After registration completes, right now the user has to restart before sugar takes into account the newly-discovered jabber server. It should immediately attempt a connection in this case.

Attachments (3)

profile.patch (569 bytes) - added by dsd 14 years ago.: prerequisite patch against sugar's profile.py for reload config functionality
presenceservice.patch (3.4 KB) - added by dsd 14 years ago.: presence service patch
schoolserver.patch (868 bytes) - added by dsd 14 years ago.: patch against sugar's hardware/schoolserver.py

Download all attachments as: .zip

Change History (28)

comment:1 Changed 14 years ago by sascha_silbe

Cc sascha_silbe added

Changed 14 years ago by dsd

Attachment profile.patch added

prerequisite patch against sugar's profile.py for reload config functionality

Changed 14 years ago by dsd

Attachment presenceservice.patch added

presence service patch

Changed 14 years ago by dsd

Attachment schoolserver.patch added

patch against sugar's hardware/schoolserver.py

comment:2 Changed 14 years ago by dsd

above patches are for sugar-0.82

comment:3 Changed 14 years ago by tomeu

Keywords r? added
Milestone changed from Unspecified by Release Team to 0.86
Priority changed from Unspecified by Maintainer to High
Severity changed from Unspecified to Major

comment:4 Changed 14 years ago by alsroot

Milestone changed from 0.86 to 0.88

changed milestone to 0.88 to not mess it with 0.86 patches

comment:5 Changed 14 years ago by martin.langhoff

Cc martin.langhoff added
Keywords ml8.2.3 added

comment:6 Changed 14 years ago by martin.langhoff

FYI - Daniel's patches are getting rolled into 8.2.2 - see http://dev.laptop.org/ticket/9726

comment:7 Changed 14 years ago by tomeu

Just to check, you don't want this applied to any branch other than 0.82?

comment:8 Changed 14 years ago by dsd

I'd like to see it shipped in all future releases of all branches. Network setup and registration is really difficult in a deployment environment and this is one step to improving the situation.

comment:9 Changed 14 years ago by martin.langhoff

Like Daniel says, I'd like to see it everywhere (with maintainers approval, etc). It is important on the first days of a deployment.

My mention of it being rolled into OLPC's 8.2.2 is a "+1 vote from downstream"; ie: "Yes, we're using this patch, it's about something end users need."

comment:10 Changed 14 years ago by rgs

Cc rgs added

comment:11 Changed 13 years ago by tomeu

Keywords r! added; r? removed

I'm taking this out from the review queue because AFAICS there are no patches that can be applied to > 0.82

comment:12 Changed 13 years ago by erikos

I guess this can be moved to 0.90 as there are no patches that can be applied to 0.88, right?

comment:13 Changed 13 years ago by erikos

Milestone changed from 0.88 to 0.90

comment:14 Changed 13 years ago by erikos

Keywords olpc-0.84 added

comment:15 Changed 13 years ago by bernie

Cc bernie added

Can someone please port these patches to 0.88?

comment:16 Changed 13 years ago by erikos

In 0.84 we added code that does restart the jabber server when we add a new one in the CP section. We should do the same when we get the information from the school server.

The patches are in #142. There is a bug that is related to that fix #1406, though.