Gaia/SMS/Scrum/2.2S13

List of bugs

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

Bugzilla link

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
1155534		[Messages][NG] Extract NewMessage view from Conversation view	---	No cf_feature-b2g	[p(2.2S13)=1][p(2.2S11)=5]	WONTFIX
1156464	Oleg Zasypkin [:azasypkin]	[Window Management] Accessing a message thread via notification after killing Messages app creates adverse effects on future notifications while in app	2.2+	No cf_feature-b2g	[3.0-Daily-Testing][p=1]	FIXED
1160232	Steve Chung [:steveck]	[SMS][Text Selection] Attempting to 'select all' in the TO: field of a new message will select text from both the TO: field and the message area	2.2+	No cf_feature-b2g	[3.0-Daily-Testing][p=1]	FIXED
1162027	Oleg Zasypkin [:azasypkin]	[Messages][New Gaia Architecture] extract inbox/index.html	---	No cf_feature-b2g	[p=1]	FIXED
1162028	Oleg Zasypkin [:azasypkin]	[Messages][NG] Extract views/conversation/conversation.html	---	No cf_feature-b2g	[p(2.2S13)=1][p=3]	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
1162031		[Meta][Messages][NG] Migrate business logic to bridge-based services	---	No cf_feature-b2g	[p=3]	WONTFIX

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

Remaining points and burndown chart

google chart api url for Sprint 2.2S13

Burndown chart

	Remaining points
Start	13
Day 2	13
Day 3
Day 4
Day 5
Day 6
Day 7
Day 8
Day 9
Day 10
Day 11
Day 12
Day 13
Day 14
Day 15
End

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

Full Query

ID	Assigned to	Summary	Blocking b2g	Feature-b2g	Whiteboard	Resolution
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

1 Total; 1 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
1162017		SMS sprint 2.2S13	---	---	INCOMPLETE
1166863	Kevin Grandon :kgrandon	[Messages] Convert string.contains to string.includes	---	---	FIXED

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

Sprint planning

Minutes are on a separate page.

Daily meetings

Day 2: 6th May

Steve

• bug 1155542 - [Messages][New Gaia Architecture] Centralizing the global components/styling into a folder
  • Patch updated with some test media path fixing(might need to create another bug for file xhr loading in assetHelper)
• bug 1156719 - [Messages][New Gaia Architecture] remove static wait screen markup and set it as shared widget
  • Might apply julien's suggestion for backward compatibility
• bug 1160232 - [SMS][Text Selection] Attempting to 'select all' in the TO: field of a new message will select text from both the TO: field and the message
  • Discuss with Jenny and she prefer to disable the recipient pill selection. Will give a quick patch today

    (Julien) what do you think? I think there could be cases where the user wants this but maybe that's edge cases? Also this should fix the other related bug 1160244.

    (Steve) I think selection for unwrapped(editing) text is enough to me, and Jenny thought selection for the wrapped element seems not really necessary as well... Maybe typing an existing contact is much easier?

    (Julien) if we can still select in an "editing" pill then I think it's ok for me.

    (Steve) Sorry not sure what's the "editing" pill?

    (Julien) I mean, what you said, the "unwrapped" :) the "contenteditable" one

    (Steve) ah, ok. That's what we propose now. And will take a look for 1160244

    (Julien) if we keep the editing pill, then bug 1160244 will still exist ;)

bug 1160261 - [SMS][Text Selection] Copy & Cutting a block of text that includes an attachment will remove the attachment but Pasting that block back will not restore it

• • Discuss with platform and UX and they don't have a clear plan for supporting the non-text selection yet. Will try to push ux if we could have an alternative solution without API support or we should just leave it.
• gaia-component stuff
  • Not much progress.

Today:

• Create subtasks for composer/conversation seperation.
• shared folder structure polishing.
• Fix the attachment menu for closing the option menu manually.

Julien

Not much yesterday besides the Sprint planning. I had a conference in the evening and had to prepare the presentation :/ I think I found a bug in our support for getUserMedia though.

(Oleg) Heh, can you point me to this bug? I'm actively using this in my side project :)

(Julien) I did not file it yet but getUserMedia doesn't capture my camera at all on master -- and also the user is not asked if the permission status is "Ask"...

(Oleg) Ah okay, I use v2.2 :)

(Julien) Ok, I haven't tried v2.2 yet, but I'll send you my test app so that you can try :) Yep! Maybe it's me !

Today: I want to:

• update wiki
• do necessary reviews
• start looking at little-browser

If all this moves forward well, I could:

• continue the prototype caching the thread list to a single db (including contacts/drafts/etc).
• I'd like to work on the deduplication logic for network alerts, it seems important for users (bug 1067938)

Oleg

• bug 1156631 - [Messages] ThreadList&Thread separation: separate CSS and image files
• Fixed last nits, got r+ and landed (landed).
• bug 1155509 - [Messages][New Gaia Architecture] Split ThreadList view from current structure
  • Resumed work on this (in progress).
• bug 1050823 - [Messages][Refactoring] Revise "ThreadUI.updateHeaderData" call while retrieving threads
  • Updated PR with two "notification" integration tests - one that runs application directly by "notification" system message and second one that fires "sms-received" system message, closes Messages, opens Utility Tray and taps on notification (in review).
• bug 1010216 - [Messages][Drafts] Improve consistency of the draft code in ThreadUI and ThreadListUI
  • Prepared first patch to get early feedback (awaiting feedback).
• bug 1156464 - [Window Management] Accessing a message thread via notification after killing Messages app creates adverse effects on future notifications while in app.
  • Investigating possible solutions:
    • If we just remove "#notification" from manifest we won't have delayed Inbox loading for this case (run app from notification);
    • If we keep the hash and remove it right before we subscribe for system message - it doesn't work for some reason, didn't investigate much though - Julien, did you try something like this in your test app?

      (Julien) nope I didn't try this. I'll add this case in my test app for bug 818000.

      (Oleg) cool, thanks, let me know if it works for you.

    • Currently I only have in mind "mozHasPendingMessages('sms-received')" additional check in startup.js to delay Inbox loading.

      (Julien) pb is that it takes time as well -- but better than nothing :/ can you try to run raptor before/after if you do this?

      (Oleg) Yeah, if it works as expected. Will need to read about running raptor locally though :)

      (Steve) I've heard this API from you few months ago but never saw anyone use it :) :)

    • Will try to find other solutions as well.

Other:

• Reviewed Steve's "move resources to shared" patch;
• Reviewed Johan's python-to-marionette-js patch (Messages-Dialer integration test);

Today:

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