Gaia/SMS/Scrum/2.2S14

List of bugs

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

Bugzilla link

Full Query
ID Assigned to Summary Blocking b2g Feature-b2g Whiteboard Resolution
1162028 Oleg Zasypkin [:azasypkin] [Messages][NG] Extract views/conversation/conversation.html --- No cf_feature-b2g [p(2.2S13)=1][p=3] FIXED
1168060 Julien Wajsberg [:julienw] [Flame][DSDS][Message]"Dial 12345 via" is displayed as heading at "Select SIM" view. --- No cf_feature-b2g [p=1] FIXED
1168441 Oleg Zasypkin [:azasypkin] [Messages][NG] Implement ActivityShim --- No cf_feature-b2g [p=3] FIXED
1169150 Steve Chung [:steveck] [Messages][NGA] Add threads.js to our codebase with script could fetch/rename the lib easily --- No cf_feature-b2g [p=1] FIXED
1169541 Steve Chung [:steveck] [Messages] [NGA] Refine the thread query and move draft query/getThreads API into conversation service / messaging service --- No cf_feature-b2g DUPLICATE
1169543 Julien Wajsberg [:julienw] [Messages][NG] Create draft spec for the main services we need --- No cf_feature-b2g [p=1] FIXED
1169558 Julien Wajsberg [:julienw] [Messages][NG] Lay out Conversation service structure --- No cf_feature-b2g [p=1] FIXED
1169573 Steve Chung [:steveck] [Messages][NG] Lay out Messaging service structure --- No cf_feature-b2g [p(FxOS-S3)=2][p(2.2S14)=1] FIXED
1169576 Oleg Zasypkin [:azasypkin] [Messages][NG] Implement Conversation service: method for streaming joined threads and drafts list --- No cf_feature-b2g [p(FxOS-S5)=3][p(FxOS-S4)=3][p(FxOS-S3)=3] FIXED
1171865 Steve Chung [:steveck] [Messages][NG] Layout mozMobileMessageShim structure --- No cf_feature-b2g [p=1] FIXED

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


Remaining points and burndown chart

google chart api url for Sprint 2.2S14

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


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

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

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
1168060 Julien Wajsberg [:julienw] [Flame][DSDS][Message]"Dial 12345 via" is displayed as heading at "Select SIM" view. --- --- FIXED
1168441 Oleg Zasypkin [:azasypkin] [Messages][NG] Implement ActivityShim --- --- FIXED
1168941 Oleg Zasypkin [:azasypkin] [Messages] Conversation view is rendered incorrectly when user tries to send message from Participants view --- --- FIXED
1169150 Steve Chung [:steveck] [Messages][NGA] Add threads.js to our codebase with script could fetch/rename the lib easily --- --- FIXED
1169255 SMS sprint 2.2S14 --- --- INCOMPLETE
1169543 Julien Wajsberg [:julienw] [Messages][NG] Create draft spec for the main services we need --- --- FIXED
1169558 Julien Wajsberg [:julienw] [Messages][NG] Lay out Conversation service structure --- --- FIXED
1169573 Steve Chung [:steveck] [Messages][NG] Lay out Messaging service structure --- --- FIXED
1171865 Steve Chung [:steveck] [Messages][NG] Layout mozMobileMessageShim structure --- --- FIXED

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


Sprint planning

Minutes are on a separate page.

Daily meetings

Day 2: 28th May

Steve

  • bug 1167495 [Messages][l10n] Fix file path changes in l10n xfail.list
    • I changed my mind to fix some l10n.get using in activity handler since it looks easy. Got some feedback from Oleg.
  • bug 1169150 - [Messages][NGA] Add threads.js to our codebase with script could fetch/rename the lib easily.
    • Add a script to download the lib and wrap up.
  • bug 1167144 - [Messages] Reduce the use of Threads.active and Threads.currentId in conversation view
    • Found we have 2 parts(exclude the before enter) that might need to access the Threads(conversationService), will raise some question within the patch.

Today:

  • Create bug for loading threads/drafts together within conversationService, I might create another one for moving mozMobileMesage api into another service first.
  • Clean up the use of threads.active in conversation view
  • Some tasks for background if I have time
    • Fix the attachment menu for closing the option menu manually
    • gaia-component stuff
    • file loading in Template.js

Julien

Note: today I'm leaving the office around 4pm, and I'll be at a conference tomorrow.

  • had the planning meeting -- still haven't created the wiki, will do it this morning.
  • reviewed the contributor bug 1153885; simple bug but I had to test thoroughly all cases
  • reviewed oleg's patch in bug 1127398 to make some code simpler and easier to refactor later
  • had a meeting with vivien (will be weekly) to explain him what we're doing and what are our issues

Today: I want to:

  • put everything in the wiki and create bugs in bugzilla
  • create a PR with initial spec
  • finish the bug 1168060 about the sim picker translation issue

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

Yesterday was hack-day, so not much done for SMS, excited by TypeScript :)

(Julien) the first time I saw TypeScript I was excited already ;)
(Oleg) Heh, I was excited more when I actually tried it :)
  • bug 1162027 - [Messages][New Gaia Architecture] extract inbox/index.html
    • Got r+, landed, it should also fix some issues that we had while running app in browser (landed).
  • bug 1162028 - [Messages][New Gaia Architecture] extract conversation/index.html
    • Moved activity service to a separate bug as was planned (no progress).
  • bug 1168441 - [Messages][NG] Implement ActivityService
    • Rebased and will rebase on Steve threads.js patch as well, will polish and add some spec related info (in progress).
  • bug 1127398 - [Messages] Display existing thread when sending a message using phone-number-/email-link context menu
    • Prepared PR, got review, extracted new part of PR to the bug below (in progress).
  • bug 1168941 - [Messages] Conversation view is rendered incorrectly when user tries to send message from Participants view
    • The bug we'll have once bug 1127398 is landed (no progress).

Other:

  • Left some feedback on Steve's l10n patch, really wondering if we should use

mozSetMessageHandlerPromise in our promisified system message handlers, but maybe not in this bug. Today:

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

Day 3: 29th May

Steve

  • bug 1167495 [Messages][l10n] Fix file path changes in l10n xfail.list
    • Landed in master

bug 1169150 - [Messages][NGA] Add threads.js to our codebase with script could fetch/rename the lib easily.

    • Get some feedback and updated
  • bug 1167144 - [Messages] Reduce the use of Threads.active and Threads.currentId in conversation view
    • No new progress yet
  • bug 1169541 - [Messages] [NGA] Refine the thread query and move draft query/getThreads API into conversation service / messaging service
    • Create a bug for refine the interface for thread query. Not sure if we need messaging service/conversation service bug ready first.

Today:

  • Start to investigate the first version of the api for query thread
  • Clean up the use of threads.active in conversation view
  • Review Oleg's patch for conversation markup splitting
  • Some tasks for background if I have time
    • Fix the attachment menu for closing the option menu manually
    • gaia-component stuff
    • file loading in Template.js

Julien

  • Absent/no report

Oleg

  • bug 1162028 - [Messages][New Gaia Architecture] extract conversation/index.html
    • Proposed monkey patch that IMO can save us some time until we know for sure how navigation will be changed/refactored (awaiting feedback).
  • bug 1168441 - [Messages][NG] Implement ActivityService
    • Rebased and the latest master and resolved conflicts. Waiting for bridge.js before asking for review (ready for review, waiting for dependency).
  • bug 1127398 - [Messages] Display existing thread when sending a message using phone-number-/email-link context menu
    • Fixed review comments and got r+, landed (landed).
  • bug 1168941 - [Messages] Conversation view is rendered incorrectly when user tries to send message from Participants view
    • Rebased on the latest master, ready for review (ready for review, will ask later).

Other:

  • Reviewed Steve's l10n patch;
  • Reviewed Steve's bridge.js patch;
  • Spent some time to understand why I couldn't run integration tests locally for some time already - turned out its problem in my new Ubuntu Gnome 15 installation (b2g wasn't run) - xhost +local: solved the problem.

Today:

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

Day 4: 1st June

Day 5: 2nd June

Day 6: 3rd June

Day 7: 4th June

Day 8: 5th June

Day 9: 8th June

Day 10: 9th June

Day 11: 10th June

Day 12: 11th June

Day 13: 12th June

Day 14: 15th June

Demos

Retrospective