Firefox OS/Comms/Dialer/Sprint/v2.1-S7/20141024-StandupNotes

From MozillaWiki
< Firefox OS‎ | Comms‎ | Dialer‎ | Sprint‎ | v2.1-S7
Jump to: navigation, search

Blockers

  • 2.1+ bug 1081854 - GAIA follow-up to bug 978639 (for GCF case 31.4.2.1.3)
    • Assigned to Tamara.
    • Depends on bug 978639, which has landed. This is ready to go.
    • Landed. Waiting for approval for uplift.
  • 2.1+ bug 1079810 - Empty sound writes into speaker, cause serious power consumption
    • Assigned to Doug.
    • Landed.
  • 2.1- bug 1082876 - TypeError: context is null at: app://callscreen.gaiamobile.org/gaia_build_defer_index.js line: 84
    • Assigned to Gabriele.
    • In reality is basically a blocker, since it’s blocking a partner’s testing, even if there are no STR.
    • Cause unknown, possibly something in shared/js/dialer/tone_player.js
  • 2.1+ bug 1082139 - JavascriptException: JavascriptException: TypeError: window.getComputedStyle(...) is null at: app://callscreen.gaiamobile.org/gaia_build_defer_index.js line: 146
    • Assigned to Germán.
    • It landed in master.
    • Uplifting to 2.1 requested.
  • 2.1+ bug 1075699 - Calling voicemail by long-pressing the 1 button displays an error and then calls
    • Assigned to Tamara.
    • Ready to land a fix, but waiting for vendor to reply to a needinfo.
    • Moved to RIL component, but we should still watch this.
    • Landed.

drs

  • Admin:
    • We got all of our 2.1+ blockers landed! Now we just have to get them approved for uplift and then do it.
  • Next steps:
    • MVC proposal.
      • This is getting a lot further. I should have it ready in a few days. It has evolved in a “design guidelines” document that includes many things, including MVC, code styling, suggested refactors, etc.
  • Patches:
    • bug 1079810 - Empty sound writes into speaker, cause serious power consumption
      • Landed.
    • bug 1083124 - [Call Log] Call log is not updated when user delete contact
      • Going to investigate this.
    • bug 1070050 - Dialer doesn't show you which # a contact called you on
      • Got review comments from Anthony. Going to iterate.
  • Reviews:
    • bug 834530 - Emergency dialer DTMF tones aren't regulated by the volume button (and are different volume from Dialer App's DTMF tones)
      • f+
    • bug 1086528 - telephony_helper_test.js incorrectly uses Promises as a way to delay a test assertion
      • r+
    • bug 1082139 - JavascriptException: JavascriptException: TypeError: window.getComputedStyle(...) is null at: app://callscreen.gaiamobile.org/gaia_build_defer_index.js line: 146
      • f+
    • bug 1072064 - [ partner confidential ]
      • r?
    • bug 1070481 - [ partner confidential ]
      • f?
  • Other:
    • In a discussion with a guy from Intel about the Telephony API.

gtorodelvalle

  • Admin:
    • As it is usual on Fridays, I won’t be able to joint the standup meeting due to family matters.
  • Next steps:
    • bug 1082139 - JavascriptException: JavascriptException: TypeError: window.getComputedStyle(...) is null at: app://callscreen.gaiamobile.org/gaia_build_defer_index.js line: 146
      • Blocker (2.1+) (see the blocker section)
    • bug 977056: [Dialer] Visual discrepancies: Call ended view under conference call participants list
      • Currently investigating why Hsin-Yi, Johan and myself are getting distinct event notification and data flows, even when Johan and myself are using exactly the same build.
      • After fixing an issue which didn’t let me get logs from the RIL (the updates to the pref.js file were not taken into consideration), I managed to modify the RIL constants file and to get these logs. I provided them to Hsin-Yi so she could try to shred some light on the issue.
    • bug 1083402 - "X has left the conference call" when a party drops off a conference call is missing:
      • The cause of the problem is the same as the one reported in bug 977056.
      • It should be solved once the patches for “bug 1059649 - [tarako/dolphin] After we hangup any one of the two calls in a group call, we'll never success to merge conference call again” are included in the generated builds. I’ll track it and check it accordingly.
      • I have not been able to test it since I am still observing an event flow which does not correspond with the ones Hsin-Yi and Johan are observing, although using the same as Johan.
      • Michael Schwartz is supposed to be having a look at it. I we do not get any reply, I’ll ping him on Monday.
    • bug 977588 - [Dialer] The Active call screen does not have the option to place calls on hold.
      • Helping Paco with this bug.
      • New version of the patch available hiding the put-on-hold and merge buttons for CDMA networks since these networks seem not to support these capabilities.

thills

    • bug 1075699 - T-Mobile Voicemail Bug. This is flagged with checkin-needed and has a fresh try run that is completed. Once this is checked in, I will land the gaia portion (though they can really be landed separately)
    • bug 1079143 - BlueTooth Bug - Posted my findings on this. Still have an NI on me for this, but prioritizing this after the blocker
    • bug 1068093 - Removed legacy call screen transitions - I’m currently trying to repro the black screen that drs is seeing. I’ve tried on Hamachi and on flame and can’t seem to repro as of yet.
    • bug 1081854 - Conference Call hangup.. The Gaia side has landed on m-c. I don’t see approval request for the gecko side to land on 2.1.
    • bug 1082193 - Review for German. We gave r+ on this one.
    • bug 1088690 - Filed this one. Narrowed it down to between 10/19 and 10/24 this occurs.

jlorenzo

    • bug 1083731: pending feedback from QA in the US to run the test with AT&T and T-Mobile.
    • bug 1086269: waiting for merge!
    • bug 1086676: waiting for second reviews
    • bug 1087926: i’m creating a test to check that the contact photo appears on the callscreen.