Support snap permissions in Discover
ClosedPublic

Authored by apol on Apr 11 2018, 12:43 AM.

Diff Detail

Repository
R134 Discover Software Store
Lint
Automatic diff as part of commit; lint not applicable.
Unit
Automatic diff as part of commit; unit tests not applicable.
apol created this revision.Apr 11 2018, 12:43 AM
Restricted Application added a project: Plasma. · View Herald TranscriptApr 11 2018, 12:43 AM
Restricted Application added a subscriber: plasma-devel. · View Herald Transcript
apol requested review of this revision.Apr 11 2018, 12:43 AM

+1 for the feature, but could we use checkboxes instead of toggles? I'm not real thrilled with the ones in those screenshots. It's not at all clear what's on and what's off.

Also those labels are not very descriptive; are there better strings available? Looking at the screenshots, I have no idea what most of those permissions actually do. A regular user would be even more confused.

apol updated this revision to Diff 31911.Apr 11 2018, 5:52 PM

Address issues by upstreap snapdqt maintainer Robert Ancell.
Change to show a check box instead of the slider.

This revision was not accepted when it landed; it landed in state Needs Review.Apr 11 2018, 11:26 PM
This revision was automatically updated to reflect the committed changes.

New in Phabricator. Is this code already merged? Using Kubuntu 18.04 with Backports PPA but I can't see these options.

Kubuntu 18.04 uses Plasma 5.12 which includes Discover 5.12. This code was added in a later version (likely 5.13).

That makes sense:

$ dpkg -l | grep discover
ii  avahi-utils                                     0.7-3.1ubuntu1.2                                    amd64        Avahi browsing, publishing and discovery utilities
ii  plasma-discover                                 5.12.8-0ubuntu0.1build1~ubuntu18.04~ppa1            amd64        Discover software management suite
ii  plasma-discover-common                          5.12.8-0ubuntu0.1build1~ubuntu18.04~ppa1            all          Discover software manager suite (common data files)
ii  plasma-discover-snap-backend                    5.12.8-0ubuntu0.1build1~ubuntu18.04~ppa1            amd64        Discover software manager suite - Snappy support

Thanks.