Skip to content
This repository has been archived by the owner. It is now read-only.

[IP-550] [Telemetry] Allow testers to set developer flag #346

Merged
merged 6 commits into from Jun 18, 2019
Merged

[IP-550] [Telemetry] Allow testers to set developer flag #346

merged 6 commits into from Jun 18, 2019

Conversation

@ghost
Copy link

@ghost ghost commented Jun 13, 2019

Add isDeveloperModeOn property to UserPreferences and if true send that as a preference to Engine. (The flag is called developer in the Engine for historical reasons.)

Update AboutSettingsTableViewController.swift to load the current status of UserPreferences.instance.isDeveloperModeOn if the hidden “More Info” mode is accessed via a two finger long press gesture.

Pull Request Checklist

  • My patch has gone through review and I have addressed review comments
  • My patch has a standard commit message that looks like Bug 12345678 - This fixes something something
  • I have updated the Unit Tests to cover new or changed functionality
  • I have updated the UI Tests to cover new or changed functionality
  • I have marked the bug with [needsuplift]
  • I have made sure that localizable strings use NSLocalizableString()

Screenshots

Screenshot 2019-06-13 at 15 28 36

Notes for testing this patch

  • Go to Settings > About
  • Perform a two finger long press gesture outside the tableview cells (e.g. on the bottom half of the screen)
  • Check if the "Developer Flag" row is shown
  • Tap the Developer Flag row to toggle developer mode to On
  • Check that the developer signal is sent
winsmith and others added 4 commits Jun 13, 2019
…nt signal

Add isDeveloperModeOn property to UserPreferences and if on, send that as a preference to Engine.

Update AboutSettingsTableViewController.swift to load the current status of UserPreferences.instance.isDeveloperModeOn if the hidden “More Info” mode is accessed via a two finger long press gesture.
Daniel Jilg
This is to prevent the Ghostery and Cliqz builds from failing, but adding the feature there as well doesn’t hurt.
Daniel Jilg
pavel-cliqz and others added 2 commits Jun 18, 2019
Daniel Jilg
@pavel-cliqz pavel-cliqz merged commit 78cec72 into ghostery:master Jun 18, 2019
3 checks passed
3 checks passed
@azure-pipelines
ghostery.browser-ios Build #20190618.7 succeeded
Details
@azure-pipelines
ghostery.browser-ios (Cliqz_IOS_CI_Build Ghostery) Cliqz_IOS_CI_Build Ghostery succeeded
Details
@azure-pipelines
ghostery.browser-ios (Cliqz_IOS_CI_Build Lumen) Cliqz_IOS_CI_Build Lumen succeeded
Details
@ghost ghost deleted the 550-dev-telemetry branch Jul 3, 2019
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Linked issues

Successfully merging this pull request may close these issues.

None yet

2 participants