Closed Bug 1251689 Opened 4 years ago Closed 4 years ago

Add rudimentary telemetry for bookmark sync state

Categories

(Firefox for iOS :: Sync, defect)

All
iOS
defect
Not set

Tracking

()

RESOLVED FIXED
Tracking Status
fxios-v3.0 --- fixed
fxios-v4.0 --- fixed

People

(Reporter: rnewman, Assigned: rnewman)

References

(Depends on 1 open bug, Blocks 2 open bugs)

Details

Attachments

(1 file)

48 bytes, text/x-github-pull-request
st3fan
: review+
st3fan
: feedback+
Details | Review
We'd like to know when users have consistent server state.

We'd like to know if they've ever been able to sync, and how long the periods are in which they cannot. (Short periods are fine.)
Blocks: 1252469
I tried to pull some of our Adjust code out so it's not so tied to Profile and Client.

Alas: ld: warning: linking against dylib not safe for use in application extensions: /Users/rnewman/moz/git/s2fxios/Carthage/Build/iOS/AdjustSdk.framework/AdjustSdk

So I'm going to build a little dropbox instead, minimize coupling.
Adjust is a bad fit here.
Summary: Add rudimentary Adjust-based telemetry for bookmark sync state → Add rudimentary telemetry for bookmark sync state
Attached file Pull req.
Assignee: nobody → rnewman
Status: NEW → ASSIGNED
Attachment #8727714 - Flags: feedback?(sarentz)
Comment on attachment 8727714 [details] [review]
Pull req.

LGTM
Attachment #8727714 - Flags: feedback?(sarentz) → feedback+
Comment on attachment 8727714 [details] [review]
Pull req.

Rebased down. Now validates even if merging is disabled.
Attachment #8727714 - Flags: review?(sarentz)
Attachment #8727714 - Flags: review?(sarentz) → review+
Blocks: 1255571
85f783c
Status: ASSIGNED → RESOLVED
Closed: 4 years ago
Resolution: --- → FIXED
Whiteboard: [needsuplift]
Target Milestone: --- → 4.0
v3.x 2de524c
Whiteboard: [needsuplift]
Depends on: 1258801
You need to log in before you can comment on or make changes to this bug.