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

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

Blockers

  • 2.1+ bug 1081714 - [Dialer][Call Screen] Phone number & name disappear from the call screen after making several calls sequentially
  • 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 is scheduled to be landed on 10/17.
    • We estimated that this will take 3 days, so it should land by 10/22.
  • 2.1+ bug 1074379 - Dialer does not always show incoming call # and prevents the user to answer
  • 2.1+ bug 1079810 - Empty sound writes into speaker, cause serious power consumption
    • Unassigned.
  • 2.1+ bug 1083729 - Workaround for the callscreen leaking
    • Landed, needs uplift.
  • 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.
    • Additional information included as the result of the investigation.
    • Info requested from Anshul.
    • We are trying to reproduce the scenario mentioned by Anshul in Telefónica creating an integration test. I have not received the results from our QA team yet. Anyhow, I hope Anshul can provide us with further information about the tests he is running.
    • It could be related to bug 548397 - window.getComputedStyle() returns null inside an iframe with display: none
  • 2.1+ bug 1075699 - Calling voicemail by long-pressing the 1 button displays an error and then calls
    • Assigned to Tamara.
    • Should lose blocker status.
    • Ready to land a fix, but waiting for vendor to reply to a needinfo.
  • 2.2+ bug 1081745 - [Dialer] "cancel" is not translated for multiple languages
    • Assigned to Germán.
    • Looks like an easy fix.
    • Not able to reproduce it. I asked Johan to try to reproduce it just to be sure.
  • 2.2+ bug 1081348 - [Dialer][Ringtone] Ringtone only plays once, does not loop, when receiving a call and will not ring on subsequent calls.
    • Unassigned.
    • Caused by bug 994190, so we need Ben Turner to investigate it.
    • Moved to DOM::IndexedDB.

drs

  • Admin:
    • Nothing new.
  • Next steps:
    • MVC proposal.
    • Reviews.
    • bug 1079810 - Empty sound writes into speaker, cause serious power consumption
  • Patches:
    • None yesterday.
  • Reviews:
    • bug 1068093 - Remove legacy transition code from the callscreen
      • r-
    • bug 1069835 - [ partner confidential ]
      • r-
    • bug 1081028 - [Dialer] Though CDMA Network doesn't support single call hold, but user can hold single call via BT Headset
      • r+
    • bug 834530 - Emergency dialer DTMF tones aren't regulated by the volume button (and are different volume from Dialer App's DTMF tones)
      • r-
    • bug 977056 - [Dialer] Visual discrepancies: Call ended view under conference call participants list
      • r?
    • bug 1080771 - Remove phone number highlighting code
      • r?
    • bug 977588 - [Dialer] The Active call screen does not have the option to place calls on hold.
      • r?
  • Other:
    • In a discussion with a guy from Intel about the Telephony API.

gtorodelvalle

  • Admin:
    • As usual on Fridays, I won’t be able to join the standup meeting, sorry! Anyhow, I included copious comments in the bugs I was working on. I also updated the bug status below .
    • Although I will probably raise this topic on Monday, it seems to me that a joint meeting between telephony back-end and front-end teams could be highly valuable at least to document the event flow and data provided as well as the errors which may be thrown. Right now this information is kind of spread amongst all of us but not documented as far as I know. In fact, https://developer.mozilla.org/en-US/docs/Web/API/Telephony is clearly outdated which highly complicates people joining our team. This issue arose as a consequence of bug 977056 (see below) but I guess it is a general feeling. On the other hand, the MVC initiative led by Doug would highly benefit from this work in my opinion :)
  • 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+ althoug I’m trying to figure out if this tag is right or it should be 2.2+).
    • bug 1081745 - [Dialer] "cancel" is not translated for multiple languages:
      • Blocker (2.2+).
    • bug 977056: [Dialer] Visual discrepancies: Call ended view under conference call participants list
      • I compiled a new build including the patches Hsin-Yi mentioned that could solve the issue regarding having 2 “connected” (mozTelephonyCall.state) calls at the same time.
      • The results obtained using that build are probably not the desired ones regarding the events and data being notified.
      • For further information, please have a look at https://bugzilla.mozilla.org/show_bug.cgi?id=977056#c43 on.
    • bug 977588 - [Dialer] The Active call screen does not have the option to place calls on hold.
      • Helping Paco with this bug.

thills

    • bug 1075699 - T-Mobile Voicemail Bug. Still no reply from partner (emailed y’day) on the NI. Will email ryang on this to get it moving.
    • bug 1079143 - BlueTooth Bug - Spent most of my time on this yesterday. Posted my findings that as soon as it’s fixed in the tree, it unearths the bug 1081714. So we might want to look at that as a potential to fix this problem.
    • bug 1068093 - Removed legacy call screen transitions - I will post a new patch for this as drs suggestion about the opacity worked great. I need to adjust the tests and then resubmit.
    • bug 1081854 - Waiting on RIL patch
    • bug 1083402 - “X has left the call does not show up”. Etienne and Carrie confirm this is a regression. Do we need to prioritize this?

jlorenzo

Manual test run status for Dialer: Total: 348 => http://mzl.la/fxos-dialer-2-1-moztrap-active-test-cases Passed: 336 Failed: 8 => http://mzl.la/fxos-dialer-2-1-moztrap-test-run-3-non-passed Invalidated: 1 => http://mzl.la/fxos-dialer-2-1-moztrap-test-run-3-non-passed Not done: 1 => https://moztrap.mozilla.org/manage/case/13733/

Retrieved from "https://wiki.mozilla.org/index.php?title=Firefox_OS/Comms/Dialer/Sprint/v2.1-S7/20141017-StandupNotes&oldid=1052197"