Confirmed users
820
edits
Gaia/SMS/Scrum/2: Difference between revisions
→Day 10: June, 6th
| Line 385: | Line 385: | ||
=== Day 10: June, 6th === | === Day 10: June, 6th === | ||
====Steve==== | |||
* {{Bug|1003820}} [Messages] Recipients container grows without limit breaking the layout | |||
** Just landed it and create {{Bug|1021513}} for follow up | |||
* {{Bug|1010690}} - [Tarako][MMS][Notification] The notification of new MMS does not appear while playing music/video in foreground | |||
** Still unable to reproduce it. I just left my comment and let QA to verify it. | |||
* {{Bug|1016897}} - [Flame][v1.4][Message]Tap new message icon, it will automatically return SMS which is just saved as draft. .And if then tap Back key, this contact will have 2 SMS items in main message view. | |||
** Seems like a problem in this.draft, maybe we should clean it while leaving the thread view. | |||
**! (Julien) yep, something like this | |||
* bug reviewing: | |||
** {{Bug|963013}}: Landed! \o/. thanks! | |||
** {{Bug|1013296}}: Send button VR, regression that makes counter indicator unable to show up was fixed in the latest patch. Need to take a look again. | |||
** {{Bug|1016541}}: Haven't started yet(but it should be quick) | |||
Today: | |||
* Fix {{Bug|1016897}} or other blocker | |||
* Clean up the review queue | |||
====Julien==== | |||
* {{Bug|1015867}}: max-height for the composer | |||
** couldn't investigate more :( | |||
* {{Bug|990537}} for DSDS refresh. | |||
** will wait for the subject handling + the new send button | |||
** won't likely be done in this sprint then :( | |||
** will do a new patch based on the 2 dependencies | |||
* {{Bug|974867}}: send email from MMS: (from madai) | |||
** reviewed this once more; I didn't try the patch yet, because some things were obviously wrong. Will probably need to review again today. | |||
* {{Bug|1007591}}: IAC communication between cost control and SMS, to silence some received SMS from the operator (TEF is doing this patch) | |||
** landed ! \o/ | |||
** interesting part: a small promise-based task runner: https://github.com/mozilla-b2g/gaia/blob/master/apps/sms/js/task_runner.js | |||
** might be a good idea to move to shared ? | |||
*:-> (Oleg) Looks like a taskQueue :) But agree can be useful for everybody | |||
*:-> (Julien) yeah it is, bad naming, ok ;) | |||
* {{Bug|992473}}: use mutation observers to retranslate DOM nodes when the l10n-id changes | |||
** found that there was actually an issue in our code (we had a wrong l10n-id that was not used currently because we don't call "translate" on the node, but their code uses it automatically), and they will remove it as part of their changes | |||
* reviewed some other small patches | |||
* made a burndown chart using the brand new camera :) | |||
*:-> (Oleg) so sad chart :( | |||
*:-> (Julien) failing is good too, will get better :) | |||
*:-> (Oleg) yeah I know that failing is learning, but anyway. We'll do better :) | |||
** https://wiki.mozilla.org/Gaia/SMS/Scrum/2#Remaining_points_and_burndown_chart | |||
Today: | |||
* definitely want to have a final patch for the max-height bug | |||
* old reviews are waiting | |||
* will try to pick up an easy blocker to cheer me up | |||
====Oleg==== | |||
* {{Bug|963013}} - [Messages][Refresh] Update bubbles style and layout of the thread. | |||
** Landed | |||
* {{Bug|1008127}} - [Messages][Refresh] Subject handling in the Composer | |||
** Updated desktop-only/mobilemessage mock, so that Pavel can test counter directly in the browser. He said that he's working on Omega's and Vicky's suggestions right now. | |||
* {{Bug|925404}} - [B2G] [SMS] Always include the phone number in the SMS Thread UI, even if the carrier is known | |||
** Have progress on it, using single template for all known for me cases: thread view header, participant view, phone header in the participant context menu, phone label in prompt when user tries to remove contact from "To" field. But there are still some places where number is shown, will ask Vicky about it. | |||
Today: will continue to work on {{Bug|925404}}, but it seems I'll carry over it to the next sprint. | |||
== Demos == | == Demos == | ||
=== {{Bug|963018}}: Composer Visual Refresh === | === {{Bug|963018}}: Composer Visual Refresh === | ||