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

From MozillaWiki
Jump to: navigation, search

Bugs

Status

Dialer v2.1-S7 burndown chart.png

  • 16,16,14,15,16,15,15,12,8,5

Bugs Taken in Planning

Full Query
ID Assigned to Summary Blocking b2g Feature-b2g Whiteboard Status Resolution
1074379 :gerard-majax Dialer does not always show incoming call # and prevents the user to answer 2.1+ No cf_feature-b2g [priority][planned-sprint][in-sprint=v2.1-S6] RESOLVED DUPLICATE
1081745 Germán Toro del Valle (:gtorodelvalle) [Dialer] "cancel" is not translated for multiple languages 2.2+ No cf_feature-b2g [2.1-exploratory-3][planned-sprint c=1] RESOLVED INVALID
1081854 Tamara Hills [:thills] GAIA follow-up to Bug 978639 (for GCF case 31.4.2.1.3) 2.0M+ No cf_feature-b2g [caf priority: p2][cr 730147][planned-sprint c=3] RESOLVED FIXED
1082139 Germán Toro del Valle (:gtorodelvalle) JavascriptException: JavascriptException: TypeError: window.getComputedStyle(...) is null at: app://callscreen.gaiamobile.org/gaia_build_defer_index.js line: 146 2.1+ No cf_feature-b2g [caf priority: p2][CR 738823][planned-sprint c=3] RESOLVED FIXED

4 Total; 0 Open (0%); 4 Resolved (100%); 0 Verified (0%);

Bugs Taken but not Completed

Full Query
ID Assigned to Summary Blocking b2g Feature-b2g Whiteboard Status Resolution
977056 Germán Toro del Valle (:gtorodelvalle) [Dialer] Visual discrepancies: Call ended view under conference call participants list --- No cf_feature-b2g [planned-sprint c=6][in-sprint=v2.1-S7] VERIFIED FIXED
977588 Paco Rampas [:paco] [Dialer] The Active call screen does not have the option to place calls on hold. --- No cf_feature-b2g kephera_53278 [planned-sprint c=][in-sprint=v2.1-S7] VERIFIED FIXED
1024341 Tamara Hills [:thills] (meta) Wake up the device after a call hangs up --- No cf_feature-b2g torch [planned-sprint][in-sprint=v2.1-S7] RESOLVED FIXED

3 Total; 0 Open (0%); 1 Resolved (33.33%); 2 Verified (66.67%);

Bugs Taken During Sprint

Full Query
ID Assigned to Summary Blocking b2g Feature-b2g Whiteboard Status Resolution
1011272 Ahmed Nefzaoui [:Nefzaoui] [B2G][RTL][Dialer] Arabic: "Call Log" in "Dialer" app has multiple issues. --- No cf_feature-b2g LocRun1.4 VERIFIED FIXED
1079810 Doug Sherk (:drs) (inactive) Empty sound writes into speaker, cause serious power consumption 2.1+ No cf_feature-b2g [caf priority: p2][CR 741215] VERIFIED FIXED
1083729 :gerard-majax Workaround for the callscreen leaking 2.1+ No cf_feature-b2g [systemsfe] RESOLVED FIXED

3 Total; 0 Open (0%); 1 Resolved (33.33%); 2 Verified (66.67%);

All Issues for this Sprint

Full Query
ID Assigned to Summary Blocking b2g Feature b2g Whiteboard Status Resolution
1011272 Ahmed Nefzaoui [:Nefzaoui] [B2G][RTL][Dialer] Arabic: "Call Log" in "Dialer" app has multiple issues. --- --- LocRun1.4 VERIFIED FIXED
1074379 :gerard-majax Dialer does not always show incoming call # and prevents the user to answer 2.1+ --- [priority][planned-sprint][in-sprint=v2.1-S6] RESOLVED DUPLICATE
1079810 Doug Sherk (:drs) (inactive) Empty sound writes into speaker, cause serious power consumption 2.1+ --- [caf priority: p2][CR 741215] VERIFIED FIXED
1081745 Germán Toro del Valle (:gtorodelvalle) [Dialer] "cancel" is not translated for multiple languages 2.2+ --- [2.1-exploratory-3][planned-sprint c=1] RESOLVED INVALID
1081854 Tamara Hills [:thills] GAIA follow-up to Bug 978639 (for GCF case 31.4.2.1.3) 2.0M+ --- [caf priority: p2][cr 730147][planned-sprint c=3] RESOLVED FIXED
1082139 Germán Toro del Valle (:gtorodelvalle) JavascriptException: JavascriptException: TypeError: window.getComputedStyle(...) is null at: app://callscreen.gaiamobile.org/gaia_build_defer_index.js line: 146 2.1+ --- [caf priority: p2][CR 738823][planned-sprint c=3] RESOLVED FIXED
1083729 :gerard-majax Workaround for the callscreen leaking 2.1+ --- [systemsfe] RESOLVED FIXED

7 Total; 0 Open (0%); 5 Resolved (71.43%); 2 Verified (28.57%);

Bugs Redirected

Full Query
ID Assigned to Summary Blocking b2g Feature b2g Whiteboard Status Resolution
1074379 :gerard-majax Dialer does not always show incoming call # and prevents the user to answer 2.1+ --- [priority][planned-sprint][in-sprint=v2.1-S6] RESOLVED DUPLICATE
1075699 Tamara Hills [:thills] Calling voicemail by long-pressing the 1 button displays an error and then calls. (Error dialog persists & must be dismissed after you hang up.) 2.1+ --- [planned-sprint][in-sprint=v2.1-S6] VERIFIED FIXED
1081745 Germán Toro del Valle (:gtorodelvalle) [Dialer] "cancel" is not translated for multiple languages 2.2+ --- [2.1-exploratory-3][planned-sprint c=1] RESOLVED INVALID
1081854 Tamara Hills [:thills] GAIA follow-up to Bug 978639 (for GCF case 31.4.2.1.3) 2.0M+ --- [caf priority: p2][cr 730147][planned-sprint c=3] RESOLVED FIXED
1082139 Germán Toro del Valle (:gtorodelvalle) JavascriptException: JavascriptException: TypeError: window.getComputedStyle(...) is null at: app://callscreen.gaiamobile.org/gaia_build_defer_index.js line: 146 2.1+ --- [caf priority: p2][CR 738823][planned-sprint c=3] RESOLVED FIXED

5 Total; 0 Open (0%); 4 Resolved (80%); 1 Verified (20%);


Daily Standup Meetings

  • Current hosts: drs, gtorodelvalle, thills
  • Hosts rotate every week.
  1. If you're not available for a standup that you're scheduled to host, then ask for someone else to host instead for just that time.
  2. Start by pinging everyone who should be participating.
  3. List any administrative items you have, and then ask for more from other people (look at the Etherpad).
  4. Look at the list of blockers and blocker nominations and see if there's anything new or that needs action. Mention these during this time.
  5. Move to individual updates. Go alphabetically, in descending order.
  6. If someone's update is taking longer than 3-4 minutes, you should generally cut them off and ask them to talk about it after the standup.
  7. Copy the reports from the GDoc to the wiki page for that day. Use the Etherpad-to-Wiki converter to format it. You can just copy and paste the whole thing and the converter will do everything for you.
  8. Ask the person who should be hosting the week after you if they'll be available. If not, move onto the next person.

Day 1: October 14

FirefoxOS/Comms/Dialer/Sprint/v2.1-S7/20141014-StandupNotes

Day 2: October 15

Blockers

  • 2.1+ bug 1081714 - [Dialer][Call Screen] Phone number & name disappear from the call screen after making several calls sequentially
    • Assigned to Germán for posterity. There is unlikely to be much/any work here.
    • This seems to be an issue with the Settings API.
    • Alexandre Lissy is investigating.
    • Fairly sure bug 1074379 is related.
  • 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
    • Assigned to Doug for posterity. There is unlikely to be much/any work here.
    • This seems to be an issue with the Settings API.
    • Alexandre Lissy is investigating.
    • Fairly sure bug 1081714 is related.
  • 2.1? bug 1082876 - TypeError: context is null at: app://callscreen.gaiamobile.org/gaia_build_defer_index.js line: 84
    • Unassigned.
    • Very likely to get blocker status.
    • 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.
    • Very likely to get blocker status.
    • Cause unknown, possibly caused by something in shared/js/dialer/font_size_manager.js.
  • 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.
  • 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.

drs

  • Admin:
    • The module peers have changed. Please see my email to the fxos-dialer mailing list.
    • Please use this Google Doc from now on for your daily reports.
    • I’m meeting with the other module owners and Wilfred today to discuss 2.2 features. Please let me know if you’d like me to ask/say anything.
    • jlorenzo is contagious, WFH tomorrow
  • Next steps:
    • MVC proposal.
    • bug 1078448 - Callscreen app instances linger in bfcache after being closed
      • Going to investigate this.
    • bug 1082876 - TypeError: context is null at: app://callscreen.gaiamobile.org/gaia_build_defer_index.js line: 84
      • Going to investigate this.
  • Patches:
    • bug 1068104 - Wake up the screen when all calls are ended
      • Backed out, going to re-land once bug 1081811 lands in a few days.
    • bug 1077331 - [System] If Vibrate is on, and a call is receive, it keeps vibrating
  • Reviews:
    • bug 977588 - [Dialer] The Active call screen does not have the option to place calls on hold.
      • r-
    • bug 977056 - [Dialer] Visual discrepancies: Call ended view under conference call participants list
      • r-
    • bug 1068093 - Remove legacy transition code from the callscreen
      • r-
  • Other:

gtorodelvalle

  • Admin:
    • Nothing to point out.
  • Next steps:
    • bug 9770056: [Dialer] Visual discrepancies: Call ended view under conference call participants list
      • I spent most of my morning including Doug’s suggestions to my patch and I found out the issue I’d like to discuss in the office hour regarding the calls being unmerged from a conference call.
    • 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. This has been added back as a blocker. Sent email to partner to get him to weigh in on Hsin-Yi's NI from a week ago.
    • bug 1079143 - BlueTooth Bug - spent fair amount of time debugging this and initially looks to be a settings issue. I NI them and I need to get more gecko logs for this so working on this.
    • bug 1068093 - Removed legacy call screen transitions - Got review- back and have some changes to make there.
    • bug 1081854 - Looks like RIL team is close to a patch on this

Day 3: October 16

Blockers

  • 2.1+ bug 1081714 - [Dialer][Call Screen] Phone number & name disappear from the call screen after making several calls sequentially
    • Assigned to Alexandre.
    • He has a fix that involves bug 1083729.
  • 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
    • Assigned to Alexandre.
    • He has a fix that involves bug 1083729.
  • 2.1? bug 1083729 - Workaround for the callscreen leaking
  • 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.
    • Info requested from Anshul.
    • 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.
  • 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.

drs

  • Admin:
    • Nothing new.
  • Next steps:
    • MVC proposal.
    • Reviews.
  • Patches:
    • None yesterday.
  • Reviews:
    • bug 977588 - [Dialer] The Active call screen does not have the option to place calls on hold.
      • r?
    • bug 1068093 - Remove legacy transition code from the callscreen
      • r-, new 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 1079794 - [Dialer] [Callscreen] When you are in active call and other call is entering you can't hold and answer with headset button
      • r-
    • bug 1083500 - Dialer needs to notify user when it can't make a call due to a problem with cardState
      • r-
    • bug 977056 - [Dialer] Visual discrepancies: Call ended view under conference call participants list
      • r-
  • Other:
    • Met with Wilfred and the other comms apps owners to talk about 2.2 features.
    • In a discussion with a guy from Intel about the Telephony API.
    • Did some of the action items from the retrospective and other meetings.

gtorodelvalle

  • Admin:
    • Jorge told me he was under some pressure in his new group and asked me to consider assigning the 3 bugs he was currently working on to other people:
      • bug 943389 - [Dialer][Conference call] Detailed contact info on conference call on group line is missing
      • bug 1029313 - [Dialer][DSDS] With 2 SIMs; The matching result disappears after typing in the last correct digit when dialing
      • bug 1062799 - Call information is not updated on the fly after creating a contact
    • Unless we consider them prioritary enough, maybe we could move them to the next sprint.
  • Next steps:
    • bug 9770056: [Dialer] Visual discrepancies: Call ended view under conference call participants list
      • I started the compilation of 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. It is taking ages and slowing my machine down a lot :( I’ll test it as soon as I have the new build.
    • bug 977588 - [Dialer] The Active call screen does not have the option to place calls on hold.
      • Helping Paco with this bug.
    • bug 1082139 - JavascriptException: JavascriptException: TypeError: window.getComputedStyle(...) is null at: app://callscreen.gaiamobile.org/gaia_build_defer_index.js line: 146
    • After investigating the issue, it seems that it could be related to “bug 548397 - window.getComputedStyle() returns null inside an iframe with display: none” and a bad configuration of the tests which are causing the issue.

thills

    • bug 1075699 - T-Mobile Voicemail Bug. Still no reply from partner (emailed y’day) on the NI. Still listed as a blocking status.
    • bug 1079143 - BlueTooth Bug - This is being listed as critical to fix now (even though we unassigned it). I’m going to look at this today and see if the 2second delay can possibly fix this because they are looking for an uplift. Need to talk about this in scrum.
    • bug 1068093 - Removed legacy call screen transitions - Posted up new changes to this.
    • 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/

Day 4: October 17

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/

Day 5: October 20

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.
  • 2.1+ bug 1079810 - Empty sound writes into speaker, cause serious power consumption
    • Assigned to Doug.
  • 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.
    • 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
    • Germán has posted a patch for this. We don’t know if it fixes the issue or not.
  • 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.

drs

  • Admin:
    • Nothing new.
  • 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
      • Looked into this a bit. I’m thinking we’re going to have to change the AudioContext initialization in the Callscreen.
  • Reviews:
    • bug 1069835 - [ partner confidential ]
      • 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-, new r?
      • Also investigated a couple of problems with this and posted my findings in the bug thread.
    • bug 1082139 - JavascriptException: JavascriptException: TypeError: window.getComputedStyle(...) is null at: app://callscreen.gaiamobile.org/gaia_build_defer_index.js line: 146
      • r?, punted to Tamara. I set myself as f?.
    • bug 1068093 - Remove legacy transition code from the callscreen
      • 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.
    • bug 977056 - [Dialer] Visual discrepancies: Call ended view under conference call participants list
      • Investigated `groupchanged` callback. We’re now discussing this with the RIL team.

thills

    • bug 1075699 - T-Mobile Voicemail Bug. Still no reply from partner. Francisco is going to help with this.
    • 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 - Need to make the changes based on latest review from drs.
    • bug 1081854 - I “integrated” the patch that was put up for review on this on Friday. I tested out the API and this seems to work. I need to do a bit of work/investigation/refreshing on the promise part of this, but I believe I should be able to post a feedback today and maybe a review tomorrow.
    • 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

  1. The manual test run is over. No changes in the number since Friday.
  2. I cleaned up the test to catch bug 1074379 (see bug 1083731). I asked QA in the US to run the test with AT&T and T-Mobile.
  3. I've set up a flashing station for Tako. I can put the base image but I still need to figure out how to properly flash gecko and gaia.

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/

gtorodelvalle

  • Absent/no report

Day 6: October 21

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.
  • 2.1+ bug 1079810 - Empty sound writes into speaker, cause serious power consumption
    • Assigned to Doug.
    • Dev Tools are holding a weak reference to the WebAudio context. Investigating more.
  • 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.
    • 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
    • Germán has posted a patch for this. We don’t know if it fixes the issue or not.
    • Anshul has confirmed the proposed patch solves the issue.
    • Review requested from Tamara ;)
  • 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.
    • We should land this as the vendor is unlikely to answer in time.

drs

  • Admin:
    • Nothing new.
  • 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
      • Dev Tools are holding a weak reference to the WebAudio context. Investigating more.
  • Reviews:
    • bug 1069835 - [ partner confidential ]
      • 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 1080771 - Remove phone number highlighting code
      • r+
    • bug 1081854 - GAIA follow-up to bug 978639 (for GCF case 31.4.2.1.3)
      • f+
  • Other:
    • In a discussion with a guy from Intel about the Telephony API.

gtorodelvalle

  • Admin:
    • Nothing to report.
  • 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
      • I provided a sequence diagram including the observed event and data flow when in a conference call amongst 3 parties one of the remote ones leaves the conference call, using master (Gecko-63faaa5.Gaia-9e8e3bb). The source code is also available and you can paste it in http://bramp.github.io/js-sequence-diagrams/ to get it depicted and also to edit it.
      • I really think it would be great to have this kind of diagrams for all the available calling scenarios and that they should be published in https://developer.mozilla.org/en-US/docs/Web/API/Telephony
      • Definitely an interesting topic for our next ‘office hour’ and/or ‘bug bash’ session.
    • bug 977588 - [Dialer] The Active call screen does not have the option to place calls on hold.
      • Helping Paco with this bug.
      • We already have a new version of the patch covering the CDMA case (disabling the put-on-hold button) waiting for Carrie’s confirmation that that is the way to go.

thills

    • bug 1075699 - T-Mobile Voicemail Bug. Still no reply from partner. Francisco is going to help with this.
    • 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 - Need to make the changes based on latest review from drs.
    • bug 1081854 - I posted a feedback for this yesterday and am currently working on tests. Running into some problems with the promises and spies so might need some help.
    • bug 1082193 - Review for German. I’ve reviewed the code, checked out the branch and tested. Found one nit, but wanted to have an IRC with drs on whether we need a test for this.
    • 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/

Day 7: October 22

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.
  • 2.1+ bug 1079810 - Empty sound writes into speaker, cause serious power consumption
    • Assigned to Doug.
    • Dev Tools are holding a weak reference to the WebAudio context. Investigating more.
  • 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.
    • 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
    • Germán will include Tamara’s comments ASAP :)
  • 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.
    • We should land this as the vendor is unlikely to answer in time.

drs

  • Admin:
    • Bug triage is tomorrow. Please let me know if you’re coming.
  • 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
      • Dev Tools are holding a weak reference to the WebAudio context. Investigating more.
    • bug 1083124 - [Call Log] Call log is not updated when user delete contact
      • Going to investigate this.
  • Reviews:
    • bug 1069835 - [ partner confidential ]
      • 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 1081854 - GAIA follow-up to bug 978639 (for GCF case 31.4.2.1.3)
      • r-
  • Other:
    • In a discussion with a guy from Intel about the Telephony API.

gtorodelvalle

  • Admin:
    • Nothing to report.
  • 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
      • It seems we currently have an agreed sequence diagram spec including the event notification and data flow for the call scenario when a participant leaves a conference call amongst 3 parties.
      • This agreed event notification and data flow should unblock the patch for this bug.
      • I really think it would be great to have this kind of diagrams for all the available calling scenarios and that they should be published in https://developer.mozilla.org/en-US/docs/Web/API/Telephony So I’ll spend some minutes on them to start collecting them.
      • Definitely an interesting topic for our next ‘office hour’ and/or ‘bug bash’ session.
    • 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.
    • bug 977588 - [Dialer] The Active call screen does not have the option to place calls on hold.
      • Helping Paco with this bug.
      • We already have a new version of the patch covering the CDMA case (disabling the put-on-hold button) waiting for Carrie’s confirmation that that is the way to go.

thills

    • bug 1075699 - T-Mobile Voicemail Bug. Re-running try and retested last night to make sure nothing changed since this sat a while and we need to land it now. Also, I responded back to the thread that this cannot be repro’d on v184 with T-Mobile. It can. I reproduced this on first try.
    • 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 - Fixed the trivial items and then posted a question back on one of the comment that still needs sorting out.
    • bug 1081854 - Conference Call hangup. Posted a review and then got review- back and reposted the PR with the changes last night.
    • bug 1082193 - Review for German. Finished this with suggestion that we should try and have a test for this.

jlorenzo

    • bug 1083731: pending feedback from QA in the US to run the test with AT&T and T-Mobile.
    • bug 1086269: one r+, waiting for the other one.
    • bug 1086676: Add a test to check notification when you miss a call from a known contact.

Day 8: October 23

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.
  • 2.1+ bug 1079810 - Empty sound writes into speaker, cause serious power consumption
    • Assigned to Doug.
    • Patch posted for Gabriele to review.
  • 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.
    • 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
    • Germán will include Tamara’s comments ASAP :)
    • Germán will include Doug’s comments ASAP :)
  • 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.
    • Set checkin-needed.

drs

  • Admin:
    • Bug triage got moved to next week.
  • 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
      • Posted a patch for Gabriele to review.
    • 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:
  • Other:
    • Had office hour yesterday. Summary is posted on fxos-dialer mailing list.
    • In a discussion with a guy from Intel about the Telephony API.

thills

    • bug 1075699 - T-Mobile Voicemail Bug. Restarted my try run on this one as they complained I did not have the right try URL for B2G.. so, using the one from the sherriffing link that they suggest as an example for B2G.
    • 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.. As of this morning, this is reopened on the Gecko side with a new patch to fix a bug that was introduced. I need to test end to end before I can land this one on the Gaia side.
    • bug 1082193 - Review for German. Commented on the fresh review. More changes coming as per drs comments.

gtorodelvalle

  • Absent or no report

jlorenzo

  • Absent or no report

Day 9: October 24

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.

Day 10: October 27

Blockers

  • 2.1-, caf+ 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.0? bug 1083124 - [Call Log] Call log is not updated when user delete contact
    • Assigned to Doug.
  • 2.0? bug 1082998 - [Midori][Call]The screen can not show a prompt"Emergency calls only"
    • Unassigned.
    • Devices team seems to be taking this.
  • 2.0? bug 834530 - Emergency dialer DTMF tones aren't regulated by the volume button (and are different volume from Dialer App's DTMF tones)
    • Assigned to Gabriele.
    • This will be a big uplift. It may also fix bug 1082876.
  • 2.2+ bug 1084252 - Airplane mode dialog not displayed when trying to call a contact with airplane mode activated
    • Assigned to Chris Lord.
    • Patch posted.

drs

  • Admin:
    • Nothing new.
  • Next steps:
    • bug 1083124 - [Call Log] Call log is not updated when user delete contact
      • Going to investigate this.
    • bug 1088668 - Dialer always new AudioContext whenever launching, cause serious power consumption
      • Going to investigate this.
  • Patches:
    • bug 1070050 - Dialer doesn't show you which # a contact called you on
      • Got review comments from Anthony. Going to iterate.
  • Reviews:
    • bug 1070481 - [woodduck][Rose][Free Test][Call log][Soul35_GEMINI]The delete Contacts Display “Eidt” UI after Enter Call log and Change Language(5/5)
      • f-
    • bug 1072064 - [Woodduck][Rose][Call log][Case Fail][SOUL35_GEMINI]The Data still displayed after you delete the call log from miss call Table.(5/5)
      • r-
  • Other:
    • Managed v2.1 uplifts and landings.
    • In a discussion with a guy from Intel about the Telephony API.

gtorodelvalle

  • Admin:
    • Nothing to report.
  • Next steps:
    • bug 977056: [Dialer] Visual discrepancies: Call ended view under conference call participants list
      • After applying Hsin-Yi’s patch for bug 1089534 - [B2G][Telephony] should GetCallFromEverywhere in Telephony::NotifyError I get the same observed results as Johan (which is great) and also closer to the specs agreed with her (https://bugzilla.mozilla.org/attachment.cgi?id=8510217).
      • This unblocks this bug on my side and I am just checking that my proposed patch is back valid ;)
    • 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.
      • After applying Hsin-Yi’s patch for bug 1089534 - [B2G][Telephony] should GetCallFromEverywhere in Telephony::NotifyError I get the same observed results as Johan and, as it was expected, this bug is solved, this is the status message informing that a participant left the conference call is properly shown.
      • I’ll track the landing of bug 1089534 - [B2G][Telephony] should GetCallFromEverywhere in Telephony::NotifyError and I will evolve this bug accordingly.
    • 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. Landed on mozilla-central. We have approval to uplift this to 2.1 and b2g34.
    • 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 - Still outstanding is the blackscreen that drs saw. I can’t repro that right now. Also, worked some on an additional test for this as per last review.
    • bug 1081854 - Conference Call hangup.. This was reverted as the 978639 was “backed out for bustage”. Need to keep eye on this one.
    • 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.

gsvelto

  • Admin:
    • Nothing new.
  • Next steps:
    • bug 834530 - Emergency dialer DTMF tones aren't regulated by the volume button (and are different volume from Dialer App's DTMF tones)
      • Patch under review, still some small fixes to do
    • bug 1082876 - TypeError: context is null at: app://callscreen.gaiamobile.org/gaia_build_defer_index.js line: 84
      • Should be fixed as part of bug 834530, some changes are needed for this though
    • bug 1031175 - [Gaia] Unify both sendMMI() and dial() functions
      • The gecko part landed and stuck, will see if I have time to work on this later this week
  • Patches:
    • bug 834530 - Emergency dialer DTMF tones aren't regulated by the volume button (and are different volume from Dialer App's DTMF tones)
      • First part was reviewed by Doug, waiting for the second part
  • Reviews:
    • None for the time being
  • Other:
    • bug 1087166 - [Dialer] Cannot switch call in CDMA network
      • Proved that this was a problem with the commercial RIL, the ball’s in the vendor court for the next steps

PTO

  • drs: Oct 13 (certain)
  • gsvelto: 4 hours on the 14th and 15th of October
  • gtorodelvalle: ?
  • jlorenzo: ?
  • paco: ?
  • thills: Monday Oct 13

Office Hours

Oct 22

Oct 15

  • Peers, team
    • Not much to be said here. Just a few clarifications on the new peer structure.
  • 2.2 Features
    • Wilfred will send out an email soon with more details.
  • MVC/Technical Direction
    • Code style, common patterns, etc. There was nothing notable here except for bug 977056 being looked over as an example of the new patterns.
  • Blockers
    • See daily meeting notes.
  • Little triage.
    • We did a mini bug triage with our remaining time.
  • Action items:
    • jlorenzo: Find the right Moztrap request to have only dialer data.

Retrospective

Retrospective Etherpad

Things we did well

  • <thills> I thought we all worked together well to get the blockers triaged.

Things we could do better

  • <drs> bug 1031175
  • <thills> I believe we should have challenged the blockiness on some of these a bit sooner than we did.
  • <gtorodelvalle> Are you aware of people or machines :p testing the Dialer and related apps apart from the unit and integration tests in Gaia? It seems to me that important bugs just show up in bunches and mostly as final users (us) use the phone to make calls.
    • <jlorenzo> QAnalyst is the team that manually tests FFOS. Here's the test suite for the Dialer (run 3 times per version) : https://moztrap.mozilla.org/manage/cases/?filter-suite=130&filter-productversion=217
    • <jlorenzo> Everybody in QA agrees, the coverage of the functional tests in Gaia need to be expanded
    • <gtorodelvalle> Did we update that coverage including the latest blockers found and solved?
    • <jlorenzo> We don't do that for now. We catch up with the poor coverage of the features. I would like to do so in the future.
  • <gsvelto> Having the Flame KK build forced upon us caused us some problems, using BT headsets is quite crash-prone now and audio controls are not working well (volume control is not working correctly)
  • <thills> I feel like a lot of things showed up late in the cycle. Maybe we can ask for %executed and the pass rate so we can help estitmae how much time we need to keep free for blockers in these sprints that are close to the end of the release.

Action items

  • <thills> Should we document our learnings about the different RILs somewhere?
  • <gtorodelvalle> Probably more coordination is needed between the QA teams since for example I am aware of a new integration test testing long calls which is only run by Telefónica. We created it during this sprint. I mean to share these contributions.
    • <jlorenzo> Ask tchung about TEF test runs.
  • <jlorenzo> Summarize the test run during the standup + have a look if doug can subscribe to the mailing for test runs.
  • <jlorenzo> Ask in the b2g-roundtable why we didn't have any communication about the KK switch
  • <gsvelto> File bugs for auditing audio channel usage.

Demos

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