Gaia/SMS/Scrum/FxOS-S8

List of bugs

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

Bugzilla link

Full Query

ID	Assigned to	Summary	Blocking b2g	Feature-b2g	Whiteboard	Resolution
1176976	Oleg Zasypkin [:azasypkin]	[Messages][Drafts] Remove the draft saving/replacing action menu	---	No cf_feature-b2g	[sms-sprint-FxOS-S8 p=1][sms-sprint-FxOS-S3][sms-sprint-FxOS-S4]	FIXED
1198266		[Messages] Use ConversationService in the application	---	No cf_feature-b2g	[sms-sprint FxOS-S8 p=1][sms-sprint FxOS-S7 p=3][sms-sprint FxOS-S6 p=2]	WONTFIX
1201016		[Messages][NG] Migrate the current Message manager event handling to NGA	---	No cf_feature-b2g	[sms-sprint FxOS-S8 p=1][sms-sprint FxOS-S7 p=1]	WONTFIX
1206678	Steve Chung [:steveck]	Outgoing MMS contains subject that was typed by user, but then hidden	2.5+	No cf_feature-b2g	[p=1]	FIXED
1207093		[Messages][Dialer] Implement low storage condition popup	---	No cf_feature-b2g	[p=2]	WONTFIX
1207094		[Messages] Disable appropriate controls when in low storage condition	---	No cf_feature-b2g	[p=2]	WONTFIX

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

Remaining points and burndown chart

google chart api url for Sprint FxOS-S8

Burndown chart

	Remaining points
Start	8
Day 2	8
Day 3	8
Day 4	8
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-FxOS-S8")

Full Query

ID	Assigned to	Summary	Blocking b2g	Feature-b2g	Whiteboard	Resolution
1176976	Oleg Zasypkin [:azasypkin]	[Messages][Drafts] Remove the draft saving/replacing action menu	---	No cf_feature-b2g	[sms-sprint-FxOS-S8 p=1][sms-sprint-FxOS-S3][sms-sprint-FxOS-S4]	FIXED
1179586	Oleg Zasypkin [:azasypkin]	Intermittent apps/sms/test/marionette/conversation_test.js | Conversation Panel Tests General use cases Long SMS thread User can see all messages when scrolls up	---	No cf_feature-b2g	[sms-sprint-FxOS-S8]	FIXED
1203886	Steve Chung [:steveck]	The back button in sms conversation view does not always work	2.5+	No cf_feature-b2g	[sms-sprint-FxOS-S8]	FIXED
1205874	Oleg Zasypkin [:azasypkin]	[Messages] After starting a new conversation from previously saved draft, that conversation will be counted twice when using Select All in Inbox	2.5+	No cf_feature-b2g	[2.5-Daily-Testing][Spark][sms-sprint-FxOS-S8]	FIXED
1208532	Oleg Zasypkin [:azasypkin]	[Messages][Refactoring] Extract SystemMessageHandler from ActivityHandler	---	No cf_feature-b2g	[sms-sprint-FxOS-S8]	FIXED

5 Total; 5 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
1176976	Oleg Zasypkin [:azasypkin]	[Messages][Drafts] Remove the draft saving/replacing action menu	---	---	FIXED
1206678	Steve Chung [:steveck]	Outgoing MMS contains subject that was typed by user, but then hidden	2.5+	---	FIXED
1207091		SMS sprint FxOS-S8	---	---	WONTFIX

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

Sprint planning

Minutes are on a separate page.

Daily meetings

Day 2: 23rd September

Steve

• bug 1179628 - [Messages][NG] Lay out Settings service structure
  • Requesting feedback, but will be in background work.
• bug 1201016 - [Messages][NG] Migrate the current Message manager event handling to NGA.
  • Waiting for feedback
• bug 1207094 - [Messages] Disable appropriate controls when in low storage condition
  • Start to test the new lowStorage flag in deviceStorage and implement the device storage(might be device storage shim).
• bug 1206678 - Outgoing MMS contains subject that was typed by user, but then hidden
  • Ni? Morpheus about how to deal with subject removal.

Today:

• Read the replay about the conversation service.
• Low storage work.

Oleg

• bug 1176976 - [Messages][Drafts] Remove the draft saving/replacing action menu
  • Got review and handling comments (in progress).
• bug 1206093 - [Messages][Tests] Leverage "client.switchToShadowRoot" to tap on gaia-header action button
  • Guiding contributor on this issue (in progress).
• bug 1179586 - Intermittent apps/sms/test/marionette/conversation_test.js | Conversation Panel Tests Action links in messages Send message to contact number
  • Investigated this one a bit, was not able to reproduce on my machine, but was more lucky on my slow gaia-vagrant setup;
  • Updated my gaia-vagrant (https://github.com/azasypkin/gaia-vagrant) script to support node v0.12 and npm v2;
  • Fixed tiny issue that I've noticed while working on gaia vagrant setup - "bug 1206687 - Python setup script points to the pip install/upgrade guide that doesn't exist anymore" :)
• bug 1206844 - Adding Firefox Emoji font support
  • Quickly looked at the PR, asked why we didn't use shared CSS var for the Emoji font - want to have closer look today.
• bug 1198266 - [Messages] Use ConversationService in the application
  • No progress (awaiting feedback).

Today:

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

Julien

• Absent/no report

Day 3: 24th September

Steve

• bug 1179628 - [Messages][NG] Lay out Settings service structure
  • Got some feedback and move to background work.
• bug 1201016 - [Messages][NG] Migrate the current Message manager event handling to NGA.
  • Might need to migrate some event handling workaround to bug 1207094.
• bug 1207094 - [Messages] Disable appropriate controls when in low storage condition
  • I think it's doable to finish the devicestorage feature as service(shim). Will reply questions later.
• bug 1206678 - Outgoing MMS contains subject that was typed by user, but then hidden
  • Still waiting for UX replies.
• bug 1203886 - The back button in sms conversation view does not always work
  • Take some time for investigation and gave a small patch.

Today:

• Read the replay about the conversation service.
• Low storage work.
• Some blockers if they have any progress.

Julien

• did a first patch for bug 1207093: popup for low storage condition (not working)
• gave feedback for stas patch for bug 1203108: conversion to l20n
• finished the review for bug 1037620 (late arrival notice) from contributor

Today: I want to:

• continue handling review/feedback/NI queue
• continue work on the low storage condition

If all this moves forward well, I could:

• continue the prototype caching the thread list to a single db (including contacts/drafts/etc).

Oleg

• bug 1176976 - [Messages][Drafts] Remove the draft saving/replacing action menu
  • Handled review comments and asked for the next round of review (in review).
• bug 1198266 - [Messages] Use ConversationService in the application
  • No progress (awaiting feedback).

Other:

• bug 1206093 - [Messages][Tests] Leverage "client.switchToShadowRoot" to tap on gaia-header action button
  • Still the same, guiding contributor on this issue (in progress).
• bug 1206844 - Adding Firefox Emoji font support
  • Trying to get more info from Pavel to better understand what it's and how we should handle it (in progress).
• bug 1179628 - [Messages][NG] Lay out Settings service structure
  • Left early feedback.
• bug 1207093 - [Messages][Dialer] Implement low storage condition popup
  • Left early feedback.
• Attended NGA meeting on behalf of Julien.

Today:

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

Day 4: 25th September

Steve

• September 28th is national holiday in Taiwan, sorry I forgot to raise it in planning...
• bug 1179628 - [Messages][NG] Lay out Settings service structure
  • Background work.
• bug 1201016 - [Messages][NG] Migrate the current Message manager event handling to NGA.
  • Migrate event handling workaround to bug 1207094.
• bug 1207094 - [Messages] Disable appropriate controls when in low storage condition
  • Migrate event handling workaround for devicestorage service(shim).
• bug 1206678 - Outgoing MMS contains subject that was typed by user, but then hidden
  • Still waiting for UX replies.
• bug 1203886 - The back button in sms conversation view does not always work
  • Create a patch for review. I tried to implement integration test but the second tap event is slower than panel switch timing. The transition logic runs in server side is much faster and tap event from client could not insert the topanel action in time.

    (Oleg) Yeah, I've left a proposition with several taps on notification that calls toPanel several times under the hood, you can try this if you have time or just left it to a separate bug :)

    (Steve) So you already implement the specific several taps action in notification integration test?

    (Oleg) We have tests and helper methods to create notifications, and have System libs to tap on them I think and we have activities as well - maybe we can wrap up something out of it :) We just need to open activity in test app while in Conversation view, create notification for the right thread, and tap several times on it, then close activity and try to go back to Inbox.

    (Steve) I wrote the navigation integration test for it already, but the problem is tap action is not fast enough to trigger the problem, even I call conversation.tap();conversation.tap(); continuously without timeout waiting.

    (Oleg) Right, that is why I thought the trick (and actually the bug QA noticed) with opened activity (while document is hidden we don't change panel, so that we can call toPanel several times) can do the job, but maybe I'm missing something.

    (Julien) otherwise maybe we can use scriptWith and call "click" twice on the link ?

    (Oleg) Hmm, maybe it will work as well, I'd try :)

Today:

• Low storage work.
• Working on some blockers if they have any progress.

Julien

• continued work on bug 1207093: popup for low storage condition, now it works on firefox, but bug 1208113 is occurring. I tried to convert it to new-style components but I don't think we should do that here. Will continue work on this today.
• gave feedback on blocker bug 1203886 about the back button that doesn't work
• gave feedback on rishav's patch for bug 1180470

Today: I want to:

• continue handling review/feedback/NI queue
• continue work on the low storage condition

Oleg

• bug 1176976 - [Messages][Drafts] Remove the draft saving/replacing action menu
  • No progress (in review).
• bug 1198266 - [Messages] Use ConversationService in the application
  • Replied to Steve's question (awaiting feedback).
• bug 1179586 - Intermittent apps/sms/test/marionette/conversation_test.js | Conversation Panel Tests General use cases Long SMS thread User can see all messages when scrolls up
  • Investigated and prepared PR that should fix intermittent failures (awaiting feedback).
• bug 1205874 - [Messages] After starting a new conversation from previously saved draft, that conversation will be counted twice when using Select All in Inbox.
  • Investigated, figured out the root cause, want to work on fix today (in progress).

Other:

• bug 1206093 - [Messages][Tests] Leverage "client.switchToShadowRoot" to tap on gaia-header action button
  • Still the same, guiding contributor on this issue (in progress).
• bug 1206844 - Adding Firefox Emoji font support
  • Reviewing, have to figure out why it partially works for me even without patch (all emojis are recognized except for the fancy numbers, they're recognized as ordinary numbers without patch), the first guess is that if some symbol can't be found in the default font it falls back to another one and eventually finds the right glyph in Emoji font, but numbers are found in the default font. Will ask Pavel (in progress).

    (Julien) emojis are complicated stuff: some symbols are in the normal unicode and part of some fonts, some other symbols are part of unicode's astral plane and part of other fonts

    (Oleg) Ugh, I thought that all emoji's symbols are not normal :) Got it, thanks for the explanation!

    (Julien) I read a very useful article some days ago about it, I'll try to find it back for you.

    (Oleg) Thanks!

    (Julien) http://eev.ee/blog/2015/09/12/dark-corners-of-unicode/

Today:

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

Day 5: 28th September

Julien

• continued work on bug 1207093: proposed a first review, was reviewed by Oleg and Gabriele already. Will answer questions and fix review comments today
  • created bug 1208456 that appears in Firefox.
  • filed a bug for the work in Settings
• reviewed blocker bug 1203886 about the back button that doesn't work
• gave feedback on bug 1179586 (test fix)
• gave another feedback on bug 1180470 (have a setting to disable sending a read report)

Today: I want to:

• continue handling review/feedback/NI queue
• continue work on the low storage condition

Oleg

• bug 1176976 - [Messages][Drafts] Remove the draft saving/replacing action menu
  • No progress (in review).

    (Julien) will look today !

    (Oleg) Thanks!

• bug 1198266 - [Messages] Use ConversationService in the application
  • No progress (awaiting feedback).
• bug 1179586 - Intermittent apps/sms/test/marionette/conversation_test.js | Conversation Panel Tests General use cases Long SMS thread User can see all messages when scrolls up
  • Got r+, landed (landed).
• bug 1205874 - [Messages] After starting a new conversation from previously saved draft, that conversation will be counted twice when using Select All in Inbox.
  • Prepared PR, asked for review (in review).
• bug 1192263 - [Messages] We load Inbox before going to the notification conversation when app is run via notification click.
  • Roughly looked at this issue, want to try fix "bug 1208532 - [Messages][Refactoring] Extract SystemMessageHandler from ActivityHandler" first if it doesn't take too much time, has some WIP (in progress).

Other:

• bug 1207093 - [Messages][Dialer] Implement low storage condition popup
  • Started to review PR, should finish soon (review in progress).
• bug 1206093 - [Messages][Tests] Leverage "client.switchToShadowRoot" to tap on gaia-header action button
  • Guided contributor, r+'ed (almost landed).
• bug 1206844 - Adding Firefox Emoji font support
  • Reviewed, PR still have issues with displaying numbers (review on hold, waiting for resolution of found issues)

Today:

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

Steve

• Absent/no report