Confirmed users
821
edits
Gaia/SMS/Scrum/FxOS-S3: Difference between revisions
/* Day 7: July 10th
(/* Remaining points and burndown chart) |
(/* Day 7: July 10th) |
||
| Line 340: | Line 340: | ||
=== Day 7: July 10th === | === Day 7: July 10th === | ||
====Steve==== | |||
* Not in office due to Typhoon, but will still online with some update as much as I can. | |||
*:(Julien) stay safe ! | |||
*:(Oleg) You should like working from home as much as I do :D | |||
* {{Bug|1169573}} - [Messages][NG] Lay out Messaging service structure | |||
** Patch update for review with test added, I also change the way of returning error because I feel the error from shim should be treat as error as well. The original way of the error return will be in service promise resolve and it seems wrong semantically. | |||
* {{Bug|1179628}} - [Messages][NG] Lay out Settings service structure | |||
** Will do {{Bug|1180593}} first | |||
* {{Bug|1180593}} - [Messages][NG] Reorganize settings methods for settings service and views separately | |||
** Start to split the getSimNameByIccId/getOperatorByIccId to view. | |||
* {{Bug|1167144}} - [Messages] Reduce the use of Threads.active and Threads currentId in conversation view. | |||
** will move to background | |||
* Review the navigation patch and an issue that failed to update the correct status. | |||
Today: | |||
* Update messaging service prototype. | |||
* Feedback for navigation work. | |||
* Settings method clean up. | |||
* Update the patch about reducing the use of Threads in conversation view. | |||
====Julien==== | |||
* especially worked on navigation ({{Bug|1162030}}): fixed easy comments, and reworked how setReady is used. | |||
* did a last review for {{Bug|1175503}} (dependencies between Compose and ConversationView) | |||
* did a last review for {{Bug|1153883}} (use of sinon in our contacts test) | |||
Today: | |||
I want to: | |||
* handle comments for the navigation patch | |||
* do reviews for sprint bugs | |||
* think about the system messages issue | |||
If all this moves forward well, I could: | |||
* continue the prototype caching the thread list to a single db (including contacts/drafts/etc). | |||
* I'd like to work on the deduplication logic for network alerts, it seems important for users ({{Bug|1067938}}) | |||
====Oleg==== | |||
* {{Bug|1155534}} - [Messages][NG] Extract NewMessage view from Conversation view | |||
** No updates (in background). | |||
* {{Bug|1175503}} - [Messages][NG] Get rid of direct ConversationView dependencies from Compose component | |||
** Got r+, landed (landed). | |||
* {{Bug|1176976}} - [Messages][Drafts] Remove the draft saving/replacing action menu | |||
** Tried to generalize related code as much as possible, have several questions, so prepared WIP patch and asked for feedback (awaiting feedback). | |||
Other: | |||
* Finished looking into and left some feedback, mostly nits and questions. | |||
Today: | |||
* Look into Julien's replies/clarifications in "{{Bug|1162030}} - [Messages][NG] Figure out how navigation works". | |||
* Will pick up next bug if I have free cycles; | |||
* Will handle review/feedback/need-info requests; | |||
* Will work on review comments and assigned bugs. | |||
=== Day 8: July 13th === | === Day 8: July 13th === | ||
=== Day 9: July 14th === | === Day 9: July 14th === | ||