Gaia/SMS/Scrum/4: Difference between revisions

Line 154: Line 154:


===Day 4: 27th June===
===Day 4: 27th June===
==== Steve ====
* {{Bug|1010690}} - [Tarako][MMS][Notification] The notification of new MMS does not appear while playing music/video in foreground
** Will verify with latest build with gecko patch {{Bug|1026737}} landed
* {{Bug|1022644}} - [Messages] Can't open the recipient panel if there are only 2 lines of recipients
** r+, will land to master and v1.3t
* {{Bug|1021513}} -  [Messages] Recipients list container scroll up automatically when dragging down the container for multiline recipients list mode
** Will create patch today. Just set the transitionend event listener on the correct element will fix this issue.
* bug reviewing:
** {{Bug|974867}} -  [MMS]Auto suggestion for email address: Still have a small issue for the multi-resolution icon.
** {{Bug|1013296}} - Compose. Change send button to an paper plane icon: Arnau update the patch again that might has less side effect on DSDS device. no update today.
** {{Bug|963043}}- [MADAI][Dialer] Select phone number from Call log  as Recipients from SMS App. Feedback given but they have to fix conflicts first.
Today:
* Try to clean(or reduce) the review queue. {{Bug|1026575}} and {{Bug|1008127}} for the first priority because of v2.0 blocker.
* Land {{Bug|1022644}} and create patch for
==== Oleg ====
* {{Bug|1008127}} - [Messages][Refresh] Subject handling in the Composer
** Tested on device, looks good to me (in review).
* {{Bug|1026575}} -  [B2G][SMS] Message preview in Messages app thread view disappears after opening app
** Prepared simple patch that doesn't respect Thread.recipients.length when auto-saving draft in the Thread panel and sent for review (in review).
* {{Bug|1010690}} - [Tarako][MMS][Notification] The notification of new MMS does not appear while playing music/video in foreground
** Tried to reproduce it on today's build (since "getSelf()" patch has been uplifted, https://bugzilla.mozilla.org/show_bug.cgi?id=1026737#c29) and at the first look I don't see issue. Maybe it needs more tries to occur, will check(ya I'm also checking with the patch uplifted, but I have no idea about the patch in https://bugzilla.mozilla.org/show_bug.cgi?id=1010690#c106 )
-> Did you try to apply it? You mean the gecko patch? No, I mean call getSelf before our own getSelf (without gecko patch)?No I haven't, and it looks weird to do so :p Yes! :) I know it won't work without gecko patch, I saw it while testing previously, when dispatchNotification is called several times in a row (all previously missed notifications)
* {{Bug|1022644}} - [Messages] Can't open the recipient panel if there are only 2 lines of recipients
** Reviewed, looks good, r+'d.
Today: will handle review comments for patches in review + going to look into next blocker we have "{{Bug|1022755}} - Possible race in the SMS navigation code"
===Day 5: 30th June===
===Day 5: 30th June===
===Day 6: 1st July===
===Day 6: 1st July===
Confirmed users
383

edits