Confirmed users
821
edits
Gaia/SMS/Scrum/2.2S5: Difference between revisions
Jump to navigation
Jump to search
→Day 2: 28th January
(Created page with "== List of bugs == <onlyinclude> === SMS issues handled by the SMS subteam (blocks the sprint {{bug|1126214}}) === [https://bugzilla.mozilla.org/buglist.cgi?f1=blocked&o1=subs...") |
|||
| Line 78: | Line 78: | ||
== Daily meetings == | == Daily meetings == | ||
=== Day 2: 28th January === | === Day 2: 28th January === | ||
====Steve==== | |||
* Spending ALL DAY with partner about the partner requirement for 2.2(Partner WW in Taipei this week) Might not be able to join the stand up in time. Will update some news if you're interested. | |||
* {{Bug|1089154}} - [Messages] investigate scoping CSS rules | |||
** Classes clean up / replaced in action menu and edit mode. Now I'm cleaning more on message app classes. Update the profile later(Still ongoing) | |||
{{Bug|1091511}} - [Flame][v2.1][Message]The number/email or URL can't be tapped if there is full-width text in this message. | |||
** Landed and ask for approval 2.1 | |||
**: (Julien) you need to ask approval for 2.2 as well :) | |||
* {{Bug|1118154}} - FFOS can not receive Cell Broadcast messages on SIM2 | |||
** Landed and ask for approval 2.1. | |||
**: (Julien) you need to ask approval for 2.2 as well :) | |||
* More performance experiment: Testing if we could render less element in the thread list(for example, delay element insertion for checkbox or contact photo) | |||
Today: | |||
* More testing replace selectors in action menu/edit mode. | |||
* Reduce element rendering experiment | |||
* code review | |||
====Julien==== | |||
* {{Bug|1089920}}: fix gaia-header performance | |||
** Continued review with wilson, found some more bugs (including in Gecko) that Wilson fixed or worked around; will land really soon now ;) | |||
* {{Bug|1082618}}: add a README file | |||
** no progress | |||
* {{Bug|1122649}}: mozSettings has a synchronous penalty | |||
** {{Bug|1115796}}'s issue didn't move forward even if I gave Fabrice all proof | |||
* {{Bug|1087981}}: remove the load penalty of notify.js | |||
** need to fix latest comments before landing | |||
* started investigating around async scripts, not much yet | |||
Today: | |||
I want to (again): | |||
* review Oleg's patches ;) | |||
* investigate if read-ahead value changes something in SMS | |||
* continue review the gaia-header patch | |||
* move forward some background matters (organize group MMS bugs, and drafts handling issues started some weeks ago) | |||
* land the README file | |||
====Oleg==== | |||
* {{Bug|1122501}} - [Messages][Drafts] Unsaved draft is silently discarded when user tries to forward message | |||
** No updates (in review) | |||
* {{Bug|1104961}} - Intermittent share_activity_test.js | Messages as share target Share via Messages Activity close button Should return to Thread panel if in Participants panel | |||
** Was looking for possible intermittent failures reasons: | |||
*** One is that we can waitElementToDisappear for element that belongs to activity window that's being disposed that - marionette is badly crashed with this, eliminated that check to fix the issue; | |||
*** The second one is recipients.js that re-renders itself several times on single "add/remove" operation. Currently don't have better idea other than timeout in test or throttle on RecipientsView.render. Still looking for better alternative (in progress). | |||
* {{Bug|1124758}} - [Messages] We should not add the hidden (+0) text at the end of the header | |||
** Working on the patch (in progress); | |||
** Prepared patch and added/updated tests for this part of functionality (in review) | |||
Today: | |||
* Will handle review/feedback/need-info requests; | |||
* Will work on review comments and assigned bugs | |||
====Teja==== | |||
*{{Bug|961572}} - [Messages][Refactoring] Implement a new independent cancel button for information view's header | |||
** Under review | |||
{{Bug|986510}} - [Messages app] Replace menu icon with settings icon when no messages in app | |||
** Fixed the bug but need to write integration tests. | |||
Today: | |||
{{Bug|986510}} - [Messages app] Replace menu icon with settings icon when no messages in app | |||
** Write integration tests and create pull request. | |||
=== Day 3: 29th January === | === Day 3: 29th January === | ||
=== Day 4: 30th January === | === Day 4: 30th January === | ||