Closed Bug 938357 Opened 11 years ago Closed 8 years ago

Secreview: Find my Device (Client-side)

Categories

(mozilla.org :: Security Assurance: Review Request, task)

x86
macOS
task
Not set
normal

Tracking

(Not tracked)

RESOLVED FIXED

People

(Reporter: pauljt, Assigned: pauljt)

References

Details

(Whiteboard: u= c= p=1 s=ready)

This review is to cover the client-side code included in FirefoxOS as part of the 'Where's my fox feature' See https://wiki.mozilla.org/Services/WheresMyFox
I presume this is the client-side parts of where is my fox:
https://github.com/dougt/whereismyfox-app
Actually no, that's the original, but has been forked. Doug can you add link, cheers?
Flags: needinfo?(doug.turner)
Whiteboard: u= c= p=1 s=ready
Flags: needinfo?(doug.turner)
Summary: Secreview: Where's my Fox (Client-side) → Secreview: Find my Device Client-side)
Summary: Secreview: Find my Device Client-side) → Secreview: Find my Device (Client-side)
This feature has since been moved to Gaia, and it's being developed at https://github.com/guilherme-pg/gaia/tree/findmydevice
thanks for the update ggp.
Paul, do we need to block on landing in mainline gaia on this review or can it happen post-landing? Thank you.
Flags: needinfo?(ptheriault)
It can happen post landing (and I can't actually finish testing until it does).

BTW is this still planned for 1.4? I assumed not  with the recent change in scope and deprioritized this review but let me know this isn't correct.
Flags: needinfo?(ptheriault)
Hey ggp,
I am starting to take a peek at the source code of findmydevice.
Whom besides you should I CC on additional bugs, that I may find during my source code review?

I will also make them block this bug for easier tracking.
I pretty much wrote all of the client code, so I'd say I'm the only person you need to CC from a technical standpoint. You may want to CC :elan on them just to make sure she's aware of the bugs we have, but I can also do that if you don't. Thanks!
Status: NEW → RESOLVED
Closed: 8 years ago
Resolution: --- → FIXED
You need to log in before you can comment on or make changes to this bug.