Gaia/SMS/Scrum/2.2S1

From MozillaWiki
< Gaia‎ | SMS‎ | Scrum
Jump to: navigation, search

List of bugs

SMS issues handled by the SMS subteam (blocks the sprint bug 1104715)

Bugzilla link

Full Query
ID Assigned to Summary Blocking b2g Feature-b2g Whiteboard Resolution
1092437 Messages app should follow text selection pattern --- No cf_feature-b2g [p=1] FIXED
1101532 Julien Wajsberg [:julienw] [Messages] load the gaia header asynchronously --- No cf_feature-b2g [sms-sprint-2.1S9] FIXED
1101619 Oleg Zasypkin [:azasypkin] [Messages][RTL] Location of status (error, delivered, read) icon at Report panel seems wrong --- No cf_feature-b2g [p=1] FIXED
1102132 Oleg Zasypkin [:azasypkin] [Messages][RTL] Move the "sliding" transition back to be the same than in LTR conditions --- No cf_feature-b2g [p=1] FIXED

4 Total; 4 Open (100%); 0 Resolved (0%); 0 Verified (0%);


Remaining points and burndown chart

google chart api url for Sprint 2.2S1

Burndown chart
Remaining points
Start 5
Day 2 5
Day 3 4
Day 4 4
Day 5
Day 6
Day 7
Day 8
Day 9
Day 10
End


SMS issues handled by the SMS subteam outside of the sprint (contains whiteboard "sms-sprint-2.2S1")

Full Query
ID Assigned to Summary Blocking b2g Feature-b2g Whiteboard Resolution
1010216 Oleg Zasypkin [:azasypkin] [Messages][Drafts] Improve consistency of the draft code in ThreadUI and ThreadListUI --- No cf_feature-b2g [sms-sprint-2.2S11][p=2] FIXED
1050823 Oleg Zasypkin [:azasypkin] [Messages][Refactoring] Revise "ThreadUI.updateHeaderData" call while retrieving threads --- No cf_feature-b2g [sms-sprint-2.2S11][p=1] FIXED
1105857 Oleg Zasypkin [:azasypkin] [Messages] User doesn't receive any notification about incoming message if native modal dialog is shown 2.0M+ No cf_feature-b2g [sms-sprint-2.2S1][sms-sprint-2.2S3] FIXED
1146393 Julien Wajsberg [:julienw] [Shinano][Aries] No attachment preview visible for pictures over 5MB --- No cf_feature-b2g [sms-most-wanted][Spark][sms-sprint-2.2S14] FIXED
1148152 Oleg Zasypkin [:azasypkin] Intermittent conversion_banner_test.js | Message Type Conversion Banner Message Type Conversion Banner for new threads MMS to email conversion and reminders --- No cf_feature-b2g [sms-sprint-2.2S11] FIXED
1155542 Steve Chung [:steveck] [Messages][New Gaia Architecture] Centralizing the global components/styling into a folder --- No cf_feature-b2g [sms-sprint-2.2S11] FIXED
1156625 Oleg Zasypkin [:azasypkin] [Messages] ThreadList&Thread separation: separate JS files --- No cf_feature-b2g [sms-sprint-2.2S11] FIXED
1156631 Oleg Zasypkin [:azasypkin] [Messages] ThreadList&Thread separation: separate CSS and image files --- No cf_feature-b2g [sms-sprint-2.2S11] FIXED
1156711 Steve Chung [:steveck] [Messages][New Gaia Architecture] remove static attachment menu markup and replace with shared widget --- No cf_feature-b2g [sms-sprint-2.2S11] FIXED
1161985 Steve Chung [:steveck] [Messages][NG] Split recipient and non-compose related styling from compose.css --- No cf_feature-b2g [sms-sprint-2.2S13] FIXED
1162030 Julien Wajsberg [:julienw] [Messages][NG] Figure out how navigation works --- No cf_feature-b2g [p(FxOS-S4)=2][p(FxOS-S3=3][p(2.2S13)=2][sms-sprint-2.2S14] FIXED
1168941 Oleg Zasypkin [:azasypkin] [Messages] Conversation view is rendered incorrectly when user tries to send message from Participants view --- No cf_feature-b2g [sms-sprint-2.2S14] FIXED

12 Total; 12 Open (100%); 0 Resolved (0%); 0 Verified (0%);


All SMS issues tracked for this sprint (target milestone)

Bugzilla link

Full Query
ID Assigned to Summary Blocking b2g Feature b2g Resolution
1053709 Steve Chung [:steveck] Make SMS messages content UI RTL-friendly --- --- FIXED
1054635 kumar rishav (:rishav_) [Messages] Context menu/long tap on container area in Composer panel causes javascript error --- --- FIXED
1072856 kumar rishav (:rishav_) [Messages] Tapping on empty space above send and attach buttons should assimilate currently edited recipient --- --- FIXED
1096330 Oleg Zasypkin [:azasypkin] [Messages][RTL] Correctly handle recipient pills --- --- FIXED
1101532 Julien Wajsberg [:julienw] [Messages] load the gaia header asynchronously --- --- FIXED
1101619 Oleg Zasypkin [:azasypkin] [Messages][RTL] Location of status (error, delivered, read) icon at Report panel seems wrong --- --- FIXED
1102132 Oleg Zasypkin [:azasypkin] [Messages][RTL] Move the "sliding" transition back to be the same than in LTR conditions --- --- FIXED
1104715 SMS sprint 2.2S1 --- --- WONTFIX

8 Total; 8 Open (100%); 0 Resolved (0%); 0 Verified (0%);


Sprint planning

Minutes are on a separate page.

Daily meetings

  • Bulleted list item

Day 2: 26th November

Oleg

  • bug 1096330 - [Messages][RTL] Correctly handle recipient pills
    • Waiting for ni?.
  • bug 1094076 - Intermittent failing test, TEST-UNEXPECTED-FAIL | apps/sms/test/marionette/composer_test.js | Messages Composer Messages Composer Test Suite Message char counter and MMS label
    • "600ms" patch was chosen (landed).
  • bug 1101619 - [Messages][RTL] Location of status (error, delivered, read) icon at Report panel seems wrong
    • Prepared patch (in review).

Other:

  • Added some Demos;
  • Usage/CostControl work

Today:

  • Will handle review/feedback requests;
  • Will work on review comments and assigned bugs

Steve

  • Absent/no report

Julien

  • Absent/no report

Day 3: 27th November

Steve

  • bug 1092437 - Messages app should follow text selection pattern
    • Create final patch for review
  • bug 1053709 - Make SMS messages content UI RTL-friendly
    • Still no reply yet.
  • About load gaia header async: The patch didn't introduce any method to prevent reflow, so the reflow penalty is still there. Although julien claimed it has obvious improvement on buri, but we are still not sure why it effect the performance... We might need to discuss this face to face in ww.
  • Some reviews, one about the report icon RTL looks good, another one about the select thread/message for read/unread might have some concern that conflict with our DOM-data decoupling idea. Will reply to Kumar on github
  • Some system issue

Today:

  • Finish the text selection patch for review.

Oleg

  • bug 1096330 - [Messages][RTL] Correctly handle recipient pills
    • Agreed to land this patch as is and handle "error" and "email" icon adjustments in the separate patch. Will land it today (checkin-needed).
  • bug 1101619 - [Messages][RTL] Location of status (error, delivered, read) icon at Report panel seems wrong
    • Got r+, land it today (checkin-needed).
  • bug 976091 - [Messages][DSDS] Show the SIM information in the message bubble
    • Prepared v2.0 patch as per request from "bug 1102706 - [FFOS2.0][SMS]SMS can't distinguish the SIM1 and SIM2 when receive SMS from the same number."

Other:

  • Yesterday was mainly working on Usage/CostControl bugs, much less on Messages.

Today:

  • Will handle review/feedback requests;
  • Will work on review comments and assigned bugs

Julien

  • Absent/no report

Day 4: 28th November

Steve

  • bug 1092437 - Messages app should follow text selection pattern
    • Will submit a patch for review later
  • bug 1053709 - Make SMS messages content UI RTL-friendly
    • Still no reply yet.
  • bug 1091550 - [Message] Unable to input any character in subject field with Simplified-Chinese IME
    • It was reproducible last month, but it seems fixed in master and even 2.1. I've asked Rudy about any possible fixing in input manager but there seems no related patch in last month. So closed as WORKSFORME first. \0/
  • Review panel sliding animation for RTL and read/unread selection patch
  • Some system issue

Today:

  • Finish the text selection patch for review.

Oleg

  • bug 1096330 - [Messages][RTL] Correctly handle recipient pills
    • Landed, filed a follow up to adjust "email" and "error" icons "bug 1105646 - [Messages] Email and Error icons inside recipient pill should be aligned in the same way".
  • bug 1101619 - [Messages][RTL] Location of status (error, delivered, read) icon at Report panel seems wrong
    • Landed.
  • bug 1102132 - [Messages][RTL] Move the "sliding" transition back to be the same than in LTR conditions
    • Prepared tiny patch (in review).
  • bug 976091 - [Messages][DSDS] Show the SIM information in the message bubble
    • Got f+, waiting for partner to test the patch (awaiting partner confirmation).
  • bug 1105129 - [FFOS2.0][Woodduck][SMS]MS can't receive the SMS until read the class0 message.
    • Investigated issue, asked partner to apply patch and check on v2.0m branch (awaiting partner feedback).
  • bug 1105857 - [Messages] User doesn't receive any notification about incoming message if native modal dialog is shown
    • While investigating bug above, noticed that we may have the same problem in other cases that seems quite serious (user will miss incoming message notifications if he forget to dismiss modal dialog).
    • Asked early feedback from Steve (awaiting feedback).
      (Steve) I haven't checked this patch yet, but it sound the same fix in bug 1105129?
      (Oleg) Yeah, I've just created PR for that patch, also it covers only class-0 message case, but it will be more or less easy to extend it if we agree on approach. I'm going to dupe bug 1105129 as it contains [woodduck] and eventually can become confidential :)

Other:

  • Usage/CostControl work.

Today:

  • Will handle review/feedback/need-info requests;
  • Will work on review comments and assigned bugs

Julien

  • Absent/no report

Day 5: 1st December

Day 6: 2nd December

Day 7: 3rd December

Day 8: 4th December

Day 9: 5th December

Day 10: 8th December

Demos

Retrospective