Gaia/SMS/Scrum/2.1S5

List of bugs

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

Bugzilla link

Full Query

ID	Assigned to	Summary	Blocking b2g	Feature-b2g	Whiteboard	Resolution
903763		[Messages] avoid css recalculations while rendering a thread	---	No cf_feature-b2g	[c=progress p= s= u=][p(2.1S5)=1]	DUPLICATE
1003843	Oleg Zasypkin [:azasypkin]	Follow up to Bug 951676 - [Messages][Refresh] Add specific image in case of more than one recipient in the list threads	---	No cf_feature-b2g	groupmms [p(2.1S6)=1][p(2.1S5)=1]	FIXED
1021608	Steve Chung [:steveck]	[Messages] Report panel visual refresh	---	No cf_feature-b2g	[sms-most-wanted][p(2.1S7)=1][p(2.1S6)=2][p(2.1S5)=2]	FIXED
1053708	Julien Wajsberg [:julienw]	Make SMS main User Interface RTL-friendly	---	No cf_feature-b2g	[p(2.1S6)=1][p(2.1S5)=2]	FIXED
1054989	Julien Wajsberg [:julienw]	[Messages][DSDS] Send button refresh	---	No cf_feature-b2g	[p=1]	FIXED
1061417	Julien Wajsberg [:julienw]	[SMS] We don't properly discard the draft in some occasions	2.1+	No cf_feature-b2g	[g+][LibGLA,TD92485,QE1, B][p=1]	FIXED
1067228	Oleg Zasypkin [:azasypkin]	[Messages][Refactoring] Move subject management to a separate component	---	No cf_feature-b2g	[p=2]	FIXED

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

Remaining points and burndown chart

google chart api url for Sprint 2.1S5

Burndown chart

	Remaining points
Start	10
Day 2	10
Day 3	10
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 (contains whiteboard "sms-sprint-2.1S5")

Full Query

ID	Assigned to	Summary	Blocking b2g	Feature-b2g	Whiteboard	Resolution
1052447	Julien Wajsberg [:julienw]	[Messages] when we're in an activity and we delete all messages in the thread, we should close the activity	2.0+	No cf_feature-b2g	[g+][LibGLA, Dev, B][sms-papercuts][sms-sprint-2.1S5]	FIXED
1061215	Oleg Zasypkin [:azasypkin]	[Messages][Refactoring] Improve handling of remaining chars counter	---	No cf_feature-b2g	[sms-papercuts][sms-sprint-2.1S5]	FIXED
1070890	Oleg Zasypkin [:azasypkin]	[Messages][Refactoring] Make use of EventDispatcher in Compose	---	No cf_feature-b2g	[sms-sprint-2.1S5]	FIXED

3 Total; 3 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
836690	Jorge Prudencio [:jorgep]	[SMS] Implement temporary in App message informing the user that they have started another SMS packet	---	---	FIXED
1052447	Julien Wajsberg [:julienw]	[Messages] when we're in an activity and we delete all messages in the thread, we should close the activity	2.0+	---	FIXED
1054989	Julien Wajsberg [:julienw]	[Messages][DSDS] Send button refresh	---	2.2+	FIXED
1061417	Julien Wajsberg [:julienw]	[SMS] We don't properly discard the draft in some occasions	2.1+	---	FIXED
1067228	Oleg Zasypkin [:azasypkin]	[Messages][Refactoring] Move subject management to a separate component	---	---	FIXED
1067820		[meta] SMS subteam sprint 2.1S5	---	---	WONTFIX

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

Sprint planning

Minutes are on a separate page.

Daily meetings

Day 2: 17th September

Julien

• I finished reading my mails and handling some bugs
• created the Scrum page; I updated the Template page (the graph URL was incorrect when the points were not 11). Maybe we could easily convert this to a script instead, now.

Today:

• will start working on the blocker bug 1061417

Oleg

• bug 1067228 - [Messages][Refactoring] Move subject management to a separate component
  • Almost finished with the main parts, today will tune and work on tests (in progress);
  • Just realized that this bug depends on "bug 1061215 - [Messages][Refactoring] Improve handling of remaining chars counter" that is currently in r? for Steve. Will rebase on master and redirect to Julien as Steve is on PTO.

Other:

• Investigated and dig through bugzilla about iframe vs unload/beforeunload. Alive asked to redirect that question to platform as it's Gecko who doesn't support that.

  (Julien) did we file a bug for having a "onclose" event for a MessagePort ?

  (Oleg) No, does spec contain that event?

  (Julien) nothing in the spec, but from what we saw we need it... we need to evolve the spec :)

  (Oleg) Ok, let's wait for Andrea reply first as start point, he is working on MessagePort and BroadcastChannel (MessageChannel too :)).

  (Julien) if we don't have an answer today, maybe send a mail to dev-webapi. Andrea can be busy :)

  (Oleg) Okay, thanks for dev-webapi reference :) I need to subscribe :)

• Added few demos to previous sprint page.

Today:

• Will handle review comments for the patches that currently in review;

Steve

• Absent/no report

Luke

• Absent/no report

Day 3: 18th September

Julien

• bug 1061417 (draft is not discarded in some occasions): it was easier than expected, and a long-standing bug probably existing since v1.4 (in review)
• resumed working on bug 1054989 (DSDS send button refresh), should have a patch ready today
• did some reviews
• looked into the strange blob-related issue that appeared while I was in holidays (bug 1059233); should have been fixed in gecko... (bug 1063658 might be this)

Other:

• resumed work on bug 874510 (upgrade mocha); one piece was missing (bug 1035133) that I added and is in review now.

Today:

• will continue working on bug 1054989
• will dig in some issues I left from yesterday

Oleg

• bug 1067228 - [Messages][Refactoring] Move subject management to a separate component
  • Still in progress (in progress);
• bug 1061215 - [Messages][Refactoring] Improve handling of remaining chars counter
  • Got first review feedback, working on it (in progress).

    (Julien) do you need something from me here? I saw a question but I don't know if you still need the answer :)

    (Oleg) I guess no :) Will ping you on IRC if have one :)

    (Julien) ok great !

• bug 1067712 - [MADAI] [SMS] Message Apps cannot remove surrogate pair character correctly
  • Investigated and provided reduced test case, now we have Core->Editor bug instead

    (Julien) I hope we don't need an uplift :/

    (Oleg) Yeah, I believe it was always there

Today:

• Will handle review comments for the patches that currently in review;
• Will handle review requests.
• Will see what events are fired on iframe in Firefox when it's removed from DOM to be sure that's not only gaia problem.

Steve

• Absent/no report

Luke

• Absent/no report