Confirmed users
383
edits
Gaia/SMS/Scrum/4: Difference between revisions
→Daily meetings
m (→Daily meetings) |
|||
| 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=== | ||