Confirmed users
821
edits
(Day 2: July 3rd) |
|||
| Line 86: | Line 86: | ||
== Daily meetings == | == Daily meetings == | ||
=== | ====Steve==== | ||
* {{Bug|1169573}} - [Messages][NG] Lay out Messaging service structure | |||
** Get some feedbacks, will reply or ask more question and update for review. | |||
* {{Bug|1179628}} - [Messages][NG] Lay out Settings service structure | |||
** Trying to summarize our thought about the Settings service (or object) | |||
* {{Bug|1167144}} - [Messages] Reduce the use of Threads.active and Threads currentId in conversation view. | |||
** Create a small patch for some early feedback. Basically want to figure out how we deal with the active thread and current thread ID once we have the conversation service. | |||
* Gave some comments about the new message extraction. | |||
Today: | |||
* Update messaging service prototype. | |||
* Reply some more thought about settings part. | |||
* Update the patch about reducing the use of Threads in conversation view. | |||
====Julien==== | |||
* gave feedback to Steve's patches, this is looking good ! | |||
* moved forward the navigation ({{Bug|1162030}}): now this is mostly working in non-split mode, today I'll work on split mode and ask feedback ! | |||
Today: | |||
I want to: | |||
* do asked reviews | |||
* move forward navigation | |||
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==== | |||
It was day of reviews :) | |||
* {{Bug|1179088}} - Stop defining non-configurable properties on windows in Gaia, again | |||
** Reviewed, r+'ed (landed). | |||
* {{Bug|1155534}} - [Messages][NG] Extract NewMessage view from Conversation view | |||
** Received feedback, replied to questions and filed smaller bugs to proceed with the split (in background). | |||
* {{Bug|1175503}} - [Messages][NG] Get rid of direct ConversationView dependencies from Compose component | |||
** Didn't have much time for this yesterday, working on unit tests (in progress). | |||
* {{Bug|1153888}} - [Messages] Contacts.findBy and other Contacts.* functions should never return "null". | |||
** Reviewed, r+'ed (landed). | |||
* {{Bug|1169573}} - [Messages][NG] Lay out Messaging service structure | |||
** Left feedback, mostly nits, otherwise looks good. | |||
* {{Bug|1179628}} - [Messages][NG] Lay out Settings service structure | |||
** Spent some time to see what we have in Settings and MobileOperator to better understand issue we have, replied at Bugzilla. | |||
* {{Bug|1178712}} - [Message]When we launch Message from a contact details view for the second time, device does not enter the compose view, but enters a view where there is only a title "message". | |||
** Didn't have time to investigate further, it's out of sprint, so maybe take a look when have some free time (in background). | |||
Today: | |||
* Will handle review/feedback/need-info requests; | |||
* Will work on review comments and assigned bugs. | |||
=== Day 3: July 6th === | === Day 3: July 6th === | ||
=== Day 4: July 7th === | === Day 4: July 7th === | ||