Confirmed users
383
edits
Gaia/SMS/Scrum/2.2S13: Difference between revisions
m
→Day 5: 11th May
| Line 252: | Line 252: | ||
=== Day 5: 11th May === | === Day 5: 11th May === | ||
====Steve==== | |||
* {{Bug|1155542}} - [Messages][New Gaia Architecture] Centralizing the global components/styling into a folder | |||
** Landed | |||
* Create subtasks for composer/conversation separation. | |||
** {{Bug|1161985}} - [Messages][NG] Split recipient and non-compose related styling from compose.css, will create a WIP for recipients.css separation | |||
** Create a WIP for clean up some duplication related to compose part. | |||
* {{Bug|1162031}} - [Messages][New Gaia Architecture] use the bridge with a shared worker | |||
** Start some experiment about the bridge lib. Before start it we might need to solve the naming conflict issue first, but I don't think our build environment will support browserify naively and we might need to browserify manually, will try to find a proper way for rename the threads module first. | |||
Today: | |||
* Solving the threads.js naming conflict temporary. | |||
* Create more subtasks for composer/conversation separation in js part. | |||
* Some tasks for background | |||
** Fix the attachment menu for closing the option menu manually | |||
** gaia-component stuff | |||
** file loading in Tamplate.js | |||
====Julien==== | |||
* Absent/no report | |||
====Oleg==== | |||
* {{Bug|1155509}} - [Messages][New Gaia Architecture] Split ThreadList view from current structure | |||
** No progress since yesterday (in progress). | |||
* {{Bug|1010216}} - [Messages][Drafts] Improve consistency of the draft code in ThreadUI and ThreadListUI | |||
** Handled most comments will ask for review today (in progress). | |||
* {{Bug|1156464}} - [Window Management] Accessing a message thread via notification after killing Messages app creates adverse effects on future notifications while in app. | |||
** Handled part of the review comments and involved Steve into discussion, my main concern is that Julien's proposition is risky for v2.2, going to prepare patch for review today once we work out good solution (in progress). | |||
* {{Bug|1127398}} - [Messages] Display existing thread when sending a message using phone-number-/email-link context menu | |||
** Rebased on the latest master since a lot of conflicting changes landed, will be ready for review once "{{Bug|1153885}} - [Messages] Remove Utils.getContactDisplayInfo" is fixed (in background). | |||
Today: | |||
* Will handle review/feedback/need-info requests; | |||
* Will work on review comments and assigned bugs. | |||
=== Day 6: 12th May === | === Day 6: 12th May === | ||
=== Day 7: 13th May === | === Day 7: 13th May === | ||