Closed
Bug 1182129
Opened 8 years ago
Closed 7 years ago
[PP] Back out privacy panel
Categories
(Firefox OS Graveyard :: Gaia::Settings, defect)
Tracking
(Not tracked)
RESOLVED
FIXED
People
(Reporter: huseby, Assigned: huseby)
References
Details
(Keywords: privacy, Whiteboard: [COM=Privacy Panel])
Attachments
(2 files, 1 obsolete file)
The time has come to back out the privacy panel. The plan is as follows: * Remove the privacy panel. * Clean up the geolocation changes, remove fuzzing, make no location, fixed location and accurate location permanent in gecko see Bug 1097229. * Maybe move the RPP out into its own app. * Merge the privacy coaching material into the existing privacy coach project and/or the FTU experience on FxOS. * Maybe merge the app permission panels with Freddy's existing verbose permissions panel.
Assignee | ||
Comment 1•8 years ago
|
||
Attachment #8632004 -
Flags: review?(etienne)
Comment 2•8 years ago
|
||
Comment on attachment 8632004 [details] [review] this patch completely backs out the privacy panel. Redirecting to a Settings peer since the only part of the patch which isn't an |rm| is there :)
Attachment #8632004 -
Flags: review?(etienne) → review?(gasolin)
Comment 3•8 years ago
|
||
@Dave could you provide more insight, discussion link, or agreement with PM so I can get some context about why we will do this?
Flags: needinfo?(huseby)
Comment 4•8 years ago
|
||
Comment on attachment 8632004 [details] [review] this patch completely backs out the privacy panel. The removal generally looks good to me, but please address above comment and set review again. Since there is a portion of verticalhome, please also include verticalhome peer in review chain.
Attachment #8632004 -
Flags: review?(gasolin) → feedback+
Comment 5•8 years ago
|
||
Thanks Paul and Wilfred to provide more information. It makes more sense then have a stand alone privacy control panel. @Dave the PR has some conflict, please fix that and set review again to me & verticalhome peer. Thanks.
Assignee | ||
Comment 7•8 years ago
|
||
rebased to HEAD
Attachment #8632004 -
Attachment is obsolete: true
Attachment #8638192 -
Flags: review?(gasolin)
Assignee | ||
Updated•8 years ago
|
Attachment #8638192 -
Flags: review?(21)
Comment 8•8 years ago
|
||
Comment on attachment 8638192 [details] [review] rebased patch to back out privacy panel Test on device and the panel is removed
Attachment #8638192 -
Flags: review?(gasolin) → review+
Comment 9•8 years ago
|
||
Since there's some discussion about privacy control's related test regression, @huseby could you rebase and merge the commit (and make sure treeherder green) soon?
Flags: needinfo?(huseby)
Comment 11•8 years ago
|
||
How's going with this issue? Dave do you still work on this?
Flags: needinfo?(huseby)
Assignee | ||
Comment 12•8 years ago
|
||
I got stuck on fixing the failing unit tests. Never figured out how to run Mullet on Linux. I had other more high priority fixes to make. I have time now to look at this again. Is there a better way to run the unit tests to fix them?
Flags: needinfo?(huseby) → needinfo?(gasolin)
Comment 13•8 years ago
|
||
As I know we use node 0.10 with npm2 for test, you could use 'nvm' to manage different versions of node and manually update to npm2. Someone also report node 0.12 works well though
Updated•8 years ago
|
Flags: needinfo?(gasolin)
Comment 14•7 years ago
|
||
pull Dave's PR and fixing tests
Comment 15•7 years ago
|
||
I think the main reason is the patch miss deleted `shared/test/unit/mocks/mock_manifest_helper.js`, so it broken other tests that used that test mock.
Comment 16•7 years ago
|
||
treeherder all green and on device test looks fine. merged to master https://github.com/mozilla-b2g/gaia/commit/87f5c9d55ab6a77dcfa48a3f3a8b4f5016f3c657
Status: NEW → RESOLVED
Closed: 7 years ago
Resolution: --- → FIXED
Comment 17•7 years ago
|
||
Should this be removed as well? http://mxr.mozilla.org/gaia/source/apps/settings/locales/settings.en-US.properties#1580 1580 privacySection.ariaLabel=Privacy Controls
Comment 18•7 years ago
|
||
thanks for catching it, fixed with https://github.com/mozilla-b2g/gaia/commit/4b47f62e10f0901f2c37f2268026fd828a26b7d5 thanks!
This is a shame we did this because: 1) there was a way to change the geolocation location ; this hurts our testing for geolocation. 2) The privacy panel was only in the engineering build and was not in production I would prefer to have it back please.
Flags: needinfo?(huseby)
Assignee | ||
Comment 20•7 years ago
|
||
we can add back a way to set the geolocation location. the rest of it needs to be refactored and merged back in if appropriate. this was backed out because it was largely broken and unmaintained code.
Flags: needinfo?(huseby)
Assignee | ||
Updated•7 years ago
|
Attachment #8638192 -
Flags: review?(21)
You need to log in
before you can comment on or make changes to this bug.
Description
•