Gaia/SMS/Scrum/4

List of bugs

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

Bugzilla link

Full Query

ID	Assigned to	Summary	Blocking b2g	Feature-b2g	Whiteboard	Resolution
990537	Julien Wajsberg [:julienw]	[DSDS] Messaging. Apply Visual Refresh to DSDS scenarios.	---	No cf_feature-b2g	[p=2]	FIXED
1021513	Steve Chung [:steveck]	[Messages] Recipients list container scroll up automatically when dragging down the container for multiline recipients list mode	2.0+	No cf_feature-b2g	[p=1]	FIXED
1022644	Steve Chung [:steveck]	[Messages] Can't open the recipient panel if there are only 2 lines of recipients	1.3T+	No cf_feature-b2g	[p=2]	FIXED
1025552	Oleg Zasypkin [:azasypkin]	[Messages][Refactoring] Refactor attachment.js and specifically move rendering part to a separate module	---	No cf_feature-b2g	[p=1]	FIXED
1026575	Oleg Zasypkin [:azasypkin]	[B2G][SMS] Message preview in Messages app thread view disappears after opening app	2.0+	No cf_feature-b2g	[2.0-daily-testing][p=2]	FIXED
1033260	Julien Wajsberg [:julienw]	[Messages] Contact suggestion list didn't dismiss when focus on subject(for all branches) and message input field(1.3t only)	---	No cf_feature-b2g	[p=1]	FIXED
1033403	Julien Wajsberg [:julienw]	[Messages] We don't stop rendering if we tap "back" too soon	---	No cf_feature-b2g	[p=1]	FIXED

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

Remaining points and burndown chart

google chart api url for Sprint 4

Burndown chart

	Remaining points
Start	7
Day 2	7
Day 3
Day 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 (blocks the sprint bug 1028276 with whiteboard "not-part-of-initial-sprint")

Full Query

ID	Assigned to	Summary	Blocking b2g	Feature-b2g	Whiteboard	Resolution
1008127	Pavel Ivanov [:ivanovpavel][:pivanov] UX	[Messages][Refresh] Subject handling in the Composer	---	No cf_feature-b2g	[sprint2 p=3][sprint3 p=2][not-part-of-initial-sprint]	FIXED
1008912	Oleg Zasypkin [:azasypkin]	'MMS' in sms app doesn't translate into other language	---	No cf_feature-b2g	[sprd309681][not-part-of-initial-sprint]	FIXED
1013296	Arnau March [:arnau] ( not working in Firefox OS anymore :( )	Compose. Change send button to an paper plane icon	---	No cf_feature-b2g	[p=1][not-part-of-initial-sprint]	FIXED
1022755	Julien Wajsberg [:julienw]	Possible race in the SMS navigation code	2.0+	No cf_feature-b2g	[p=3][not-part-of-initial-sprint]	FIXED
1030160	Steve Chung [:steveck]	[Messages][MMS] Subject is considered empty (placeholder is displayed) even if it has several empty lines	2.0+	No cf_feature-b2g	[not-part-of-initial-sprint]	FIXED
1034637	Oleg Zasypkin [:azasypkin]	[Messages] Utils.js unit tests are failing when run locally	---	No cf_feature-b2g	[not-part-of-initial-sprint]	FIXED

6 Total; 6 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
990537	Julien Wajsberg [:julienw]	[DSDS] Messaging. Apply Visual Refresh to DSDS scenarios.	---	2.0	FIXED
1008127	Pavel Ivanov [:ivanovpavel][:pivanov] UX	[Messages][Refresh] Subject handling in the Composer	---	---	FIXED
1008912	Oleg Zasypkin [:azasypkin]	'MMS' in sms app doesn't translate into other language	---	---	FIXED
1013296	Arnau March [:arnau] ( not working in Firefox OS anymore :( )	Compose. Change send button to an paper plane icon	---	---	FIXED
1021513	Steve Chung [:steveck]	[Messages] Recipients list container scroll up automatically when dragging down the container for multiline recipients list mode	2.0+	---	FIXED
1022644	Steve Chung [:steveck]	[Messages] Can't open the recipient panel if there are only 2 lines of recipients	1.3T+	---	FIXED
1022755	Julien Wajsberg [:julienw]	Possible race in the SMS navigation code	2.0+	---	FIXED
1025552	Oleg Zasypkin [:azasypkin]	[Messages][Refactoring] Refactor attachment.js and specifically move rendering part to a separate module	---	---	FIXED
1026575	Oleg Zasypkin [:azasypkin]	[B2G][SMS] Message preview in Messages app thread view disappears after opening app	2.0+	---	FIXED
1030160	Steve Chung [:steveck]	[Messages][MMS] Subject is considered empty (placeholder is displayed) even if it has several empty lines	2.0+	---	FIXED

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

Sprint planning

Minutes are on a separate page.

Daily meetings

Day 2: 25th June

Steve

• bug 1010690 - [Tarako][MMS][Notification] The notification of new MMS does not appear while playing music/video in foreground
  • Partner reported that the issue still exist, but I could not reproduce it...
• bug 1022644 - [Messages] Can't open the recipient panel if there are only 2 lines of recipients
  • Feedback+, still working on the test
• bug reviewing:
  • bug 925404 - [B2G] [SMS] Always include the phone number in the SMS Thread UI, even if the carrier is known: r=me
  • bug 1025552 - Refactoring for attachment rendering: No serious issue discovered right now, maybe we can land for this sprint.
  • bug 974867 - [MMS]Auto suggestion for email address: Some suggestion gave, the test still failed.
  • bug 959201 - [Messages][Drafts] Wrong Cursor position in the message compose of a Draft: Some feedback given and r+

Today:

• Try to clean(or reduce) the review queue
• Update patch in bug 1022644

Oleg

• bug 008127 - [Messages][Refresh] Subject handling in the Composer
  • We finally have patch that works, and don't see any issues! It requires JS changes, but it's much less risky now. So I prefer to go with it instead of changing layout and fixing regressions later on. Tightly collaborated with Pavel yesterday, it speeds up the process a lot. Some cleanup is left from both CSS and JS sides, once we finish, Pavel will ask for review from Steve (in progress, almost done).
• bug 1025552 - [Messages][Refactoring] Refactor attachment.js and specifically move rendering part to a separate module
  • Rebased on the latest master (in review).
• bug 1026575 - [B2G][SMS] Message preview in Messages app thread view disappears after opening app
  • Tried to reproduce it many times, with Buri and Flame, using git revisions mentioned in the bug and master, but without luck. Noticed that in video attached to the bug, timestamps for the thread and the single message in it are different that made me think that thread was manipulated somehow, but QA didn't confirm that. The only a bit similar case that I found is when we have "empty" draft for the thread (just a space or new line in message input an save it as a draft), but it's still not the same issue.

-> Steve, do you have any ideas? I'm using PVT builds, but QA mentioned some tinderbox builds, I don't really know much about it, found only something similar (https://pvtbuilds.mozilla.org/pvt/mozilla.org/b2gotoro/tinderbox-builds/). Can "tinderbox vs nightly pvt" builds be the reason why I can't repro this? -> Based on QA's description, tinderbox build will produce more frequently (4 times a day or more) for finding the regression window with less effort. But basically the code base is the same. The only possibility might be the some changes that need patches from both gaia and gecko, but tinderbox build created with one side is not ready yet. But if it's still reproducible on the latest build, maybe we could ask them to try in on PVT build? -> Yeah, probably it's the only thing I can do :) Thanks! Will ask Jayme to check that with PVT. And will try to manually flash with tinderbox build they mentioned.

• bug 1013296 - Compose. Change send button to an paper plane icon
  • Went through the bug and haven't noticed why it can depend on subject handling patch, we may touch the same stuff in CSS, but it isn't really a dependency. Asked Arnau to proceed with this patch.

Other:

• • Added demos to the Sprint 3 Demo page for the carrier header patch + closed related "bug 883911 - [SMS][MMS] Update all occurrences of "Type ? Carrier ? Number" strings to same format" as it's covered by the carrier header patch.

Today: handle review comments for attachment refactoring patch, will polish JS part for the subject handling patch.