Opened 10 years ago

Last modified 4 years ago

#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 10 years ago.
prerequisite patch against sugar's profile.py for reload config functionality
presenceservice.patch (3.4 KB) - added by dsd 10 years ago.
presence service patch
schoolserver.patch (868 bytes) - added by dsd 10 years ago.
patch against sugar's hardware/schoolserver.py

Download all attachments as: .zip

Change History (28)

comment:1 Changed 10 years ago by sascha_silbe

  • Cc sascha_silbe added

Changed 10 years ago by dsd

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

Changed 10 years ago by dsd

presence service patch

Changed 10 years ago by dsd

patch against sugar's hardware/schoolserver.py

comment:2 Changed 10 years ago by dsd

above patches are for sugar-0.82

comment:3 Changed 10 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 10 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 10 years ago by martin.langhoff

  • Cc martin.langhoff added
  • Keywords ml8.2.3 added

comment:6 Changed 10 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 10 years ago by tomeu

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

comment:8 Changed 10 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 10 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 9 years ago by rgs

  • Cc rgs added

comment:11 Changed 9 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 9 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 9 years ago by erikos

  • Milestone changed from 0.88 to 0.90

comment:14 Changed 9 years ago by erikos

  • Keywords olpc-0.84 added

comment:15 Changed 9 years ago by bernie

  • Cc bernie added

Can someone please port these patches to 0.88?

comment:16 Changed 9 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.

comment:17 Changed 9 years ago by erikos

The above comment is for 0.84-0.88. 0.90 is another story...

comment:18 Changed 9 years ago by erikos

Tomeu said, that he would look into the 0.90 issue (so we would need to 'reset' the config in the CP and in schoolserver.py).

comment:19 Changed 6 years ago by dnarvaez

  • Milestone changed from 0.90 to 1.0

I wonder if this ever landed in master.

comment:20 Changed 6 years ago by dnarvaez

Doesn't seem to be landed.

comment:21 Changed 6 years ago by dnarvaez

  • Milestone changed from 1.0 to Unspecified

comment:22 Changed 6 years ago by dnarvaez

  • Bug Status changed from Unconfirmed to New

comment:23 Changed 5 years ago by godiard

  • Priority changed from High to Normal

comment:24 Changed 5 years ago by godiard

  • Priority changed from Normal to Low

comment:25 Changed 4 years ago by godiard

  • Keywords patch added; r! ml8.2.3 olpc-0.84 removed
Note: See TracTickets for help on using tickets.