Ticket #7544 (closed enhancement: fixed)

Opened 5 years ago

Last modified 5 years ago

[sugar-toolkit] Add a "Publish" option to sugar profile

Reported by: gdesmott Owned by: marco
Priority: normal Milestone: 9.1.0-cancelled
Component: sugar Version: not specified
Keywords: r+ Cc:
Action Needed: finalize Verified: no
Deployments affected: Blocked By:
Blocking:

Description

This ticket is to track Gadget integration in sugar-toolkit.

See http://wiki.laptop.org/go/Gadget and related pages for more information about Gadget.

We need to add a "publish" option in the Jabber section of sugar's profile. If this option is set then PS will register to Gadget when connected.

Change History

  Changed 5 years ago by gdesmott

This is implemented in this patch: https://dev.laptop.org/git?p=users/guillaume/sugar-toolkit;a=commitdiff;h=406598a780e0b4ba30c2ffb3ce8821f651f8e892

  Changed 5 years ago by marco

  • keywords r? added
  • next_action changed from never set to review
  • milestone changed from 8.2.0 (was Update.2) to 9.1.0

  Changed 5 years ago by gdesmott

Would be cool if a sugar dev could review this patch as I'd like to merge it at the beginning of the 9.1.0 cycle.

  Changed 5 years ago by gdesmott

This should be done using gconf now. This branch add a the needed key: https://dev.laptop.org/git?p=users/guillaume/sugar;a=shortlog;h=7544

  Changed 5 years ago by gdesmott

The patch adding the gconf key was merged. We still need to add code using it before closing this bug.

follow-up: ↓ 7   Changed 5 years ago by gdesmott

https://dev.laptop.org/git?p=users/guillaume/sugar;a=shortlog;h=7544 has been updated and now publish information to Gadget when discovered if needed.

in reply to: ↑ 6   Changed 5 years ago by gdesmott

Replying to gdesmott:

https://dev.laptop.org/git?p=users/guillaume/sugar;a=shortlog;h=7544 has been updated and now publish information to Gadget when discovered if needed.

reviewed by Simon and merged.

The last part is to watch the gconf key and call Publish when it's changed. But I can't do that now as sugar-control-panel is broken.

  Changed 5 years ago by gdesmott

  • spec_stage set to unknown
  • spec_reviewed set to 0

I added a gconf watcher but for some reason, the callback is not called when the key is changed. Help welcome. https://dev.laptop.org/git?p=users/guillaume/sugar;a=shortlog;h=7544

  Changed 5 years ago by erikos 

self.gconf_client.add_dir('/desktop/sugar/user', gconf.CLIENT_PRELOAD_NONE)

the directory you want to preload is '/desktop/sugar/collaboration/'

  Changed 5 years ago by gdesmott

Of course, I'm an idiot. Sorry.

I fixed my branch which is now working.

  Changed 5 years ago by erikos

  • keywords r+ added; r? removed

Looks good. You don't need to cache default client - it is cached in the lib - the API is not nice that you have to request the client first when interacting. Anyhow if you cache it please make it private -> self._gconf_client.

  Changed 5 years ago by gdesmott

I removed the cache.

  Changed 5 years ago by gdesmott

  • status changed from new to closed
  • next_action changed from review to finalize
  • resolution set to fixed

Thanks for the review. Merged to master.

Note: See TracTickets for help on using tickets.