3,860
edits
m (→Day 7: July 1) |
m (Lakrits moved page FirefoxOS/Comms/Dialer/Sprint/1 to Firefox OS/Comms/Dialer/Sprint/1: The official spelling of "Firefox OS" leaves a space between the two parts of the name. It's easier to find a page if the spelling of its name is consistent...) |
||
(7 intermediate revisions by one other user not shown) | |||
Line 7: | Line 7: | ||
"target_milestone": "2.0 S5 (4July)", | "target_milestone": "2.0 S5 (4July)", | ||
"whiteboard": "planned-sprint", | "whiteboard": "planned-sprint", | ||
"whiteboard_type": "contains", | |||
"include_fields": "id, assigned_to, summary, cf_blocking_b2g, cf_feature-b2g, whiteboard, status, resolution" | |||
} | |||
</bugzilla> | |||
* Does not include [[#Bugs Taken but not Completed|Bugs Taken but not Completed]]. | |||
====Bugs Taken but not Completed==== | |||
<bugzilla> | |||
{ | |||
"component": "Gaia::Dialer", | |||
"whiteboard": "in-sprint=v2.0-S5", | |||
"whiteboard_type": "contains", | "whiteboard_type": "contains", | ||
"include_fields": "id, assigned_to, summary, cf_blocking_b2g, cf_feature-b2g, whiteboard, status, resolution" | "include_fields": "id, assigned_to, summary, cf_blocking_b2g, cf_feature-b2g, whiteboard, status, resolution" | ||
Line 29: | Line 39: | ||
} | } | ||
</bugzilla> | </bugzilla> | ||
* Does not include [[#Bugs Taken but not Completed|Bugs Taken but not Completed]]. | |||
====Blockers Without an Assignee==== | ====Blockers Without an Assignee==== | ||
<bugzilla> | <bugzilla> | ||
Line 55: | Line 66: | ||
=Daily Meetings= | =Daily Meetings= | ||
{{:FirefoxOS/Comms/Dialer/Standup Guidelines}} | {{:FirefoxOS/Comms/Dialer/Standup Guidelines}} | ||
==Day 10: July 4== | |||
{{:FirefoxOS/Comms/Dialer/Sprint/1/20140704-Minutes|Meeting Minutes}} | |||
====drs==== | |||
* Next steps: {{Bug|1029581}} - [Dialer] Contact name is not shown in the call log after having two contacts with same number and the first one is removed | |||
** I'm going to take this today. | |||
** Nevermind, Anthony took it. | |||
* I also want to start thinking more about long-term work that we can be doing since we're starting to really stabilize. In particular, I'm ironing out our estimations process and thinking about long-term projects. | |||
* {{Bug|1017365}} - 1.4+ The "Emergency number" is displayed as "Emergency nu...": | |||
** Waiting for Germán to update this to not have the wallpaper changes, then will review. | |||
* {{Bug|1032502}} - 2.0+ The phone label of an incoming call on the lockscreen is not translated. | |||
** This is a really gross uplift requiring some build hacks. Yuren is helping me out with this. | |||
* {{Bug|1030546}} - 1.4+ [Emergency call] "Cancel" button is not localized | |||
** This is a really gross uplift requiring some build hacks. Yuren is helping me out with this. | |||
* {{Bug|1021478}} - Set outgoing calls "Always ask" with two sim card inside, use bluetooth headset to dial, there will be no network message | |||
** Need to review this for Wei | |||
====Rik==== | |||
* Helped a contributor with tablet contribution {{bug|1032282}} | |||
* Tried to help Gabriele a bit and found a bug in the process: {{bug|1034279}} | |||
* Fixed broken tests: {{bug|1032784}} | |||
* Discussed MMI API design with RIL team : {{bug|1031193}} | |||
* Started looking at {{bug|1029581}} | |||
====gsvelto==== | |||
* {{Bug|1016885}} - [Tarako][Dolphin][LMK][Youtube][USSD] Pop-up USSD message causes Youtube being killed at background due to LMK while using Youtube app | |||
** The fix landed on master this morning, I'm working on the v1.3t patch which should be ready soon enough. This hasn't been nom'd for v1.4 but I have the feeling it will. | |||
*** Rik: IMHO, when a bug is greenlit for a branch and has also landed on master, it should land on all branches up to the one it was greenlit for. We've been bitten by {{bug|990003}} because of that. So yup, we should land it on 1.4. | |||
** (updated July 3rd) I landed a follow-up because of an issue on multi-SIM devices I had missed in the original patch. I've got informal approval from release management to land on 1.4 so I'll land there too. Totally agree on landing on all branches starting with the older one that got approval but hey, release management doesn't always seem to agree :-( | |||
** (updated July 4th) v1.3t patch is almost ready. It required some significant changes and even a fix for a bizarre race condition in the LazyL10n code. I've asked for v1.4 approval for the dependent bug and I intend to land this one on v2.0 too. | |||
* {{Bug|990003}} - [Dolphin][Tarako][Perf][Dialer] It takes a long time for the call screen shows up | |||
** I had uplifted two dependencies and whipped together an almost working uplift; Etienne offered to help me out on that one to finish the uplifts. | |||
** (Updated July 3rd) Etienne is taking care of the final uplifts | |||
* {{Bug|1002327}} - We are trying to get imei by dialing the number *#06# on a device, which has two sim card slots, but only one imei returns | |||
** No work done yet, will be starting today | |||
** (Updated July 3rd) No progress here yet, too busy with the other bug | |||
** (Updated July 4th) Finally got my Flame build working, if I'm not too tired I'll try to fix this one later today | |||
====gtorodelvalle==== | |||
* {{Bug|1018283}} - [Follow-up 951665] Pending visual revision and adjustments of the VR call screen when in lockscreen: | |||
** Waiting for Travis to be all green until another criteria is set for landing. | |||
** Yesterday I suffered from the problems with the tests failing in master since only at a late extent I noticed it was not caused by the latest version of my patch. | |||
** I had to ask Fernando Jiménez several times to relaunch some TBPL tasks. I just did a couple of minutes ago since there was a notification sending test failing. | |||
* {{Bug|1017365}} - [V1.4][Dialer]The "Emergency number" is displayed as "Emergency nu...": | |||
** Discussing with Vicky, Anthony and Doug the fix to be implemented and the way to do it ;) | |||
** Sent a new pull request implementing Doug's approach but setting the wallpaper as the background as requested by Vicky. | |||
*** Rik: I think we've been clear that the background change will NOT land in this bug. If Visual wants the background change, they need to open a bug and nominate it for 1.4. | |||
** Review asked from Doug. | |||
** Just noticed Anthony's comment so I am preparing a new version of the patch without the wallpaper fix and asked Vicky to fill a new bug for it. | |||
* {{Bug|1010104}} - [Dialer][Call Screen] Baseline of the contact name when applying the fluid font size: | |||
** I started working on this patch and I will send Vicky a proposal today (July, the 2nd) to try to decided on the final solution to be implemented. | |||
** I showed Vicky a prototype and just received Vicky's go ahead so going ahead :p | |||
** Launched the pull request and waiting for the test checking. Once they pass, I'll ask for review. | |||
** Waiting for Travis. TBPL is fine with the patch. | |||
* Other issues: | |||
** I spend part of the day helping Paco with his unstable environment regarding tests. Seems to be more stable now :) | |||
====etienne==== | |||
* {{Bug|990003}} and dependencies landed on 1.4, brassing for a potential backout | |||
* Still some back and forth on Gabriele's USSD stuff, fun times | |||
====thills==== | |||
* {{Bug|1021550}}. need to retest on the emulator now since my new test had one tweak in the final r+ from Hsin-Yi that I want to make sure. Right now, emulator build is not working. | |||
* {{Bug|1030550}} - Working on trying to find the root cause for this. I can repro this on the Hamachi. | |||
**Gecko device Build issue resolved -- Thanks drs. | |||
==Day 9: July 3== | |||
{{:FirefoxOS/Comms/Dialer/Sprint/1/20140703-Minutes|Meeting Minutes}} | |||
====drs==== | |||
* Next steps: {{bug|1029581}} - [Dialer] Contact name is not shown in the call log after having two contacts with same number and the first one is removed | |||
** Currently assigned to Dietrich, but he hasn't worked on it for a while. I think someone should take this, so I will if nobody else wants to. | |||
* I also want to start thinking more about long-term work that we can be doing since we're starting to really stabilize. In particular, I'm ironing out our estimations process and thinking about long-term projects. | |||
* {{bug|1017365}} - 1.4+ The "Emergency number" is displayed as "Emergency nu...": | |||
** Need to review this for Germán | |||
* {{bug|1032502}} - 2.0+ The phone label of an incoming call on the lockscreen is not translated. | |||
** Landed on master, but I'm not sure how this is going to get uplifted. | |||
* {{bug|1030546}} - 1.4+ [Emergency call] "Cancel" button is not localized | |||
** Landed on master, but I'm not sure how this is going to get uplifted. | |||
* {{bug|1021478}} - Set outgoing calls "Always ask" with two sim card inside, use bluetooth headset to dial, there will be no network message | |||
** Need to review this for Wei | |||
====Rik==== | |||
* Wednesday is my "Gaia quality afternoon" so not much done on Dialer | |||
* Reviews in progress, no bug assigned, I can help on your bugs | |||
** drs: Should we move {{bug|1030550}} to Tamara? | |||
====gsvelto==== | |||
* {{bug|1016885}} - [Tarako][Dolphin][LMK][Youtube][USSD] Pop-up USSD message causes Youtube being killed at background due to LMK while using Youtube app | |||
** The fix landed on master this morning, I'm working on the v1.3t patch which should be ready soon enough. This hasn't been nom'd for v1.4 but I have the feeling it will. | |||
*** Rik: IMHO, when a bug is greenlit for a branch and has also landed on master, it should land on all branches up to the one it was greenlit for. We've been bitten by {{bug|990003}} because of that. So yup, we should land it on 1.4. | |||
** (updated July 3rd) I landed a follow-up because of an issue on multi-SIM devices I had missed in the original patch. I've got informal approval from release management to land on 1.4 so I'll land there too. Totally agree on landing on all branches starting with the older one that got approval but hey, release management doesn't always seem to agree :-( | |||
* {{bug|990003}} - [Dolphin][Tarako][Perf][Dialer] It takes a long time for the call screen shows up | |||
** I had uplifted two dependencies and whipped together an almost working uplift; Etienne offered to help me out on that one to finish the uplifts. | |||
** (Updated July 3rd) Etienne is taking care of the final uplifts | |||
* {{bug|1002327}} - We are trying to get imei by dialing the number *#06# on a device, which has two sim card slots, but only one imei returns | |||
** No work done yet, will be starting today | |||
** (Updated July 3rd) No progress here yet, too busy with the other bug | |||
====gtorodelvalle==== | |||
* {{bug|1018283}} - [Follow-up 951665] Pending visual revision and adjustments of the VR call screen when in lockscreen: | |||
** Waiting for Travis to be all green until another criteria is set for landing. | |||
* {{bug|1017365}} - [V1.4][Dialer]The "Emergency number" is displayed as "Emergency nu...": | |||
** Discussing with Vicky, Anthony and Doug the fix to be implemented and the way to do it ;) | |||
** Sent a new pull request implementing Doug's approach but setting the wallpaper as the background as requested by Vicky. | |||
*** Rik: I think we've been clear that the background change will NOT land in this bug. If Visual wants the background change, they need to open a bug and nominate it for 1.4. | |||
** Review asked from Doug. | |||
* {{bug|1031257}} - User can't see "multicall" name in the upper information panel: | |||
** I started working on this bug but when testing it many call screen issues arose which seem to be related to {{bug|1031227}} I spent some time with Fernando González Sela providing him with the logs he needed. | |||
** Seems to be solved and everything back to normal. Loli will re-check tomorrow (3/7). | |||
* {{bug|1010104}} - [Dialer][Call Screen] Baseline of the contact name when applying the fluid font size: | |||
** I started working on this patch and I will send Vicky a proposal today (July, the 2nd) to try to decided on the final solution to be implemented. | |||
** I showed Vicky a prototype and just received Vicky's go ahead so going ahead :p | |||
** Launched the pull request and waiting for the test checking. Once they pass, I'll ask for review. | |||
====etienne==== | |||
* {{bug|990003}} and dependencies are ready for uplift (waiting on a QA round) | |||
* Looking at a intermittent Audio channel issue caused by the TonePlayer ({{bug|1029979}}) | |||
* Finished reviewing Gabriele's USSD notification patch ({{bug|1016885}}) | |||
====thills==== | |||
* {{bug|1021550}}. Received an r+ from Hsin-Yi on this bug. Need to find someone with level 3 permissions to help check in. | |||
** drs: You can add the checkin-needed keyword on Bugzilla, or I can land it for you. | |||
* {{bug|1030550}} Started looking at this and trying to repro with Hamachi. I tried the workaround with my BT set that does not have the conf feature. | |||
I'm generally struggling with build issues on B2G right now that have taken up a big amount of time. | |||
** drs: What in particular? | |||
==Day 8: July 2== | |||
{{:FirefoxOS/Comms/Dialer/Sprint/1/20140702-Minutes|Meeting Minutes}} | |||
====drs==== | |||
* Next steps: Find more blockers to work on. It looks like most of the unassigned ones have been taken and/or fixed. As usual, let me know if you need help with something. | |||
* {{bug|1017365}} - 1.4+ The "Emergency number" is displayed as "Emergency nu...": | |||
** Proposed an alternative that Germán implemented, now sitting in review. I plan on getting to this in a few hours. | |||
* {{bug|1032502}} - 2.0+ The phone label of an incoming call on the lockscreen is not translated. | |||
** Patch posted, review? to Francisco. This is a pretty simple l10n issue. I didn't understand that the l10n files are located offsite though and I made a mistake because of this, so I'll have to fix that. | |||
* {{bug|1030546}} - 2.0+ [Emergency call] "Cancel" button is not localized | |||
** Patch posted, ~review+ from Etienne. This is a pretty simple l10n issue. I didn't understand that the l10n files are located offsite though and I made a mistake because of this, so I'll have to fix that. | |||
* {{bug|1021478}} - Set outgoing calls "Always ask" with two sim card inside, use bluetooth headset to dial, there will be no network message | |||
** Review-, redirected to Carrie for UX question. This isn't a blocker so it's low priority. | |||
* {{bug|1033127}} - it will always ask user to select sim before calling in call log even if the default sim for outgoing call had been set yet | |||
** Feedback-. I don't think this is actually a problem and this bug is in triage for 1.4? status. | |||
* {{bug|1019783}} - Output from *#06# on FLAME DIALER used to return 2 IMEI | |||
** I was unable to repro the desired functionality which was supposedly broken some time ago. gsvelto and I believe that they may have been using different firmware. gsvelto is going to write a fix for this even though we don't believe that it's a regression. | |||
====Rik==== | |||
* drs: see {{bug|1017365}} please, I would like your feedback on my proposal | |||
** I'd prefer needinfos, I'm not checking this before the meeting | |||
* {{bug|1030550}} - Callscreen app sometimes calls BT command "CHLD=3" twice when it only receive one system message | |||
** It is probably a Gecko issue, needinfoed Gene on it. | |||
====gsvelto==== | |||
* {{bug|1016885}} - [Tarako][Dolphin][LMK][Youtube][USSD] Pop-up USSD message causes Youtube being killed at background due to LMK while using Youtube app | |||
** The fix landed on master this morning, I'm working on the v1.3t patch which should be ready soon enough. This hasn't been nom'd for v1.4 but I have the feeling it will. | |||
* {{bug|990003}} - [Dolphin][Tarako][Perf][Dialer] It takes a long time for the call screen shows up | |||
** I had uplifted two dependencies and whipped together an almost working uplift; Etienne offered to help me out on that one to finish the uplifts. | |||
* {{bug|1002327}} - We are trying to get imei by dialing the number *#06# on a device, which has two sim card slots, but only one imei returns | |||
** No work done yet, will be starting today | |||
====gtorodelvalle==== | |||
* Weren't we suppose to update the wiki instead of using Etherpad? I do not see updates at [[FirefoxOS/Comms/Dialer]] ? Am I pointing to the wrong place? Thanks! | |||
** drs: We moved back to Etherpad because people prefer its ease of use. Also, we can discuss things like I'm doing now. | |||
* {{bug|1018283}} - [Follow-up 951665] Pending visual revision and adjustments of the VR call screen when in lockscreen: | |||
** Waiting for Travis to be all green until another criteria is set for landing. | |||
* {{bug|1017365}} - [V1.4][Dialer]The "Emergency number" is displayed as "Emergency nu...": | |||
** Discussing with Vicky, Anthony and Doug the fix to be implemented and the way to do it ;) | |||
** Sent a new pull request implementing Doug's approach but setting the wallpaper as the background as requested by Vicky. | |||
** Review asked from Doug. | |||
* {{bug|1031257}} - User can't see "multicall" name in the upper information panel: | |||
** I started working on this bug but when testing it many call screen issues arose which seem to be related to {{bug|1031227}} I spent some time with Fernando González Sela providing him with the logs he needed. | |||
** Seems to be solved and everything back to normal. Loli will re-check tomorrow (3/7). | |||
* {{bug|1010104}} - [Dialer][Call Screen] Baseline of the contact name when applying the fluid font size: | |||
** I started working on this patch and I will send Vicky a proposal today (July, the 2nd) to try to decided on the final solution to be implemented. | |||
** I showed Vicky a prototype and just received Vicky's go ahead so going ahead :p | |||
====etienne==== | |||
* Quick l10n related review for Doug | |||
* Long review comment on {{bug|1018494}} introducing Paco to the integration tests | |||
* Working the on the uplift from hell ({{bug|990003}}) | |||
====thills==== | |||
* {{bug|1021550}}. I received feedback this morning from Hsin-Yi on this patch and had some minor issues to fix. I fixed those this morning and reposted for her review again. | |||
* Next Steps: finish of {{bug|1021550}} once I get the r+ from Hsin-Yi. It looks like all the blockers are taken or waiting on someone else, so I will go back to working on bug 1016698 unless there is something higher priority (need feedback from team on this). | |||
** drs: Rik suggested taking {{bug|1030550}}. | |||
==Day 7: July 1== | ==Day 7: July 1== | ||
Line 86: | Line 282: | ||
==Day 6: June 30== | ==Day 6: June 30== | ||
{{:FirefoxOS/Comms/Dialer/Sprint/1/20140630-Minutes|Meeting Minutes}} | |||
====drs==== | ====drs==== |
edits