Ticket #1151 (new defect)

Opened 4 years ago

Last modified 3 years ago

connect to gabble immediately after registration

Reported by: dsd Owned by: tomeu
Priority: High Milestone: 0.90
Component: sugar Version: Unspecified
Severity: Major Keywords: r!, ml8.2.3, olpc-0.84
Cc: sascha_silbe, martin.langhoff, rgs, bernie Distribution/OS: Unspecified
Bug Status: Unconfirmed

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

profile.patch Download (0.6 KB) - added by dsd 4 years ago.
prerequisite patch against sugar's profile.py for reload config functionality
presenceservice.patch Download (3.4 KB) - added by dsd 4 years ago.
presence service patch
schoolserver.patch Download (0.8 KB) - added by dsd 4 years ago.
patch against sugar's hardware/schoolserver.py

Change History

Changed 4 years ago by sascha_silbe

  • cc sascha_silbe added

Changed 4 years ago by dsd

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

Changed 4 years ago by dsd

presence service patch

Changed 4 years ago by dsd

patch against sugar's hardware/schoolserver.py

Changed 4 years ago by dsd

above patches are for sugar-0.82

Changed 4 years ago by tomeu

  • keywords r? added
  • priority changed from Unspecified by Maintainer to High
  • severity changed from Unspecified to Major
  • milestone changed from Unspecified by Release Team to 0.86

Changed 4 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

Changed 4 years ago by martin.langhoff

  • cc martin.langhoff added
  • keywords ml8.2.3 added

Changed 4 years ago by martin.langhoff

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

Changed 3 years ago by tomeu

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

Changed 3 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.

Changed 3 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."

Changed 3 years ago by rgs

  • cc rgs added

Changed 3 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

Changed 3 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?

Changed 3 years ago by erikos

  • milestone changed from 0.88 to 0.90

Changed 3 years ago by erikos

  • keywords r!, ml8.2.3, olpc-0.84 added; r! ml8.2.3 removed

Changed 3 years ago by bernie

  • cc bernie added

Can someone please port these patches to 0.88?

Changed 3 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.

Changed 3 years ago by erikos

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

Changed 3 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).

Note: See TracTickets for help on using tickets.