Closed Bug 937240 Opened 8 years ago Closed 8 years ago

[COST CONTROL] Remove the `mindgap.js` module and filtering operations of historical data.


(Firefox OS Graveyard :: Gaia::Cost Control, defect)

Not set


(Not tracked)

1.3 Sprint 5 - 11/22


(Reporter: salva, Assigned: mai)




(1 file, 1 obsolete file)

With the new Netowrk Statistics API, mindgap.js module is no longer required. It must be removed in order to fully unlock all the features provided by the new API.
blocking-b2g: --- → 1.3?
Assignee: nobody → mri
Target Milestone: --- → 1.3 Sprint 5 - 11/22
Attached file proposal patch v1 (obsolete) —
Please, could you review my code?
There is a pending issue: testing of the application with two different SIM cards
Attachment #832280 - Flags: review?(salva)
Comment on attachment 832280 [details]
proposal patch v1

See some comments on GitHub. It's going well but I want to discuss some aspects. Provide your opinion if you don't share the point of view of any of the comments.

When ready, ask for review again.

Thank you :mai!
Attachment #832280 - Flags: review?(salva)
Comment on attachment 832280 [details]
proposal patch v1

Updated the code keeping in mind your comments
Attachment #832280 - Flags: review?
Attachment #832280 - Flags: review? → review?(salva)
Comment on attachment 832280 [details]
proposal patch v1

Nice work with lots of improvements. I'm happy to see mindgap.js is gone!

Please don't merge until bug 937041 is in Mozilla Central.

Thank you :mai, a really good work!

NOTE: I made a little name fix and change `availableNetworks()` into `getAvailableNetworks()` because this is the ultimate name for the method.
Attachment #832280 - Flags: review?(salva) → review+
Depends on: 937041
Master: 445a9fb8deddd15e617f000ef208bddbedd28f0b
Closed: 8 years ago
Resolution: --- → FIXED
Duplicate of this bug: 937247
already in master. clear flag
blocking-b2g: 1.3? → ---
Depends on: 941716
This just broke the utility tray entirely.

John - Can you back this out?
Flags: needinfo?(jhford)
Please, John, before remove this, let's see if we can solve this by providing a simple patch. Jason, when the bug 928325 broke the utility tray and this was reported in bug 939110, it was not backed out. I am not asking for a special treatment, but note these kind of things happens. Let's find a solution before taking light decisions.
Hi. Root cause found. Our fault, backing out:

My apologies if the previous comment sounded rude. I only wanted to see what was more convenient: if a patch or another PR. Given the situation, it is better to make a backout: better to track one commit rather than two. :)
blocking-b2g: --- → 1.3?
Resolution: FIXED → ---
Attached file v1.1
Attachment #832280 - Attachment is obsolete: true
Attachment #8336727 - Flags: review?(salva)
Flags: needinfo?(jhford)
Comment on attachment 8336727 [details]

Added two harmless modifications in a couple of lines not affecting the behaviour but the code style.

Thanks for the fix, :mai!
Attachment #8336727 - Flags: review?(salva) → review+
Master: 84a0570132f6af235000f08ef02b6798cbfa2d0d
Closed: 8 years ago8 years ago
Resolution: --- → FIXED
already in master v1.3. clear flag
blocking-b2g: 1.3? → ---
Depends on: 964502
No longer depends on: 964502
You need to log in before you can comment on or make changes to this bug.