Gaia/SMS/Scrum/2.2S14

From MozillaWiki
Jump to navigation Jump to search

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 14
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

Steve

  • bug 1169150 - [Messages][NGA] Add threads.js to our codebase with script could fetch/rename the lib easily.
    • R+ and need to move out the script from tool to sms root for bower component
  • bug 1169541 - [Messages] [NGA] Refine the thread query and move draft query/getThreads API into conversation service / messaging service
    • Ongoing, but will land the messaging service layout first for getThreads(or necessity of this api in messaging or conversation service)
  • bug 1169573 - [Messages][NG] Lay out Messaging service structure
    • Will give a patch for simply layout today. Basically the API is almost the same with getThreads/getMessages kept inside. We can discuss if we really need this in the bug
  • bug 1167144 - [Messages] Reduce the use of Threads.active and Threads.currentId in conversation view
    • No new progress yet

Today:

  • Land bridge lib today
  • Give a patch for messaging service and 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

  • received my Aries so I learnt how to flash it - now I need to decide if I want master for dogfood ;)
  • I had a look at how exiftran rotates jpeg images losslessly for bug 1169055 (I left links there if you want to have a look)
  • thought a little in bug 1146393: we should put the limit higher now that we use moz-sample-size - maybe 10MB is good? That would be really not risky (unless the limit is too high).
  • quickly reviewed bug 1127398 that refactors how forward/activity work
  • finished bug 1168060 - latest patch is really simple ;)
  • quickly reviewed bug 1169150 from Steve about the shell script to add threads.js - sorry to be picky ;)
  • added some information in bug 1040874 for a contributor to pick it up
  • in the train, rebased the db experiment with latest master to make it easier to look at: https://github.com/mozilla-b2g/gaia/pull/28681 (even if it's not in the sprint, I think it's good to at least keep it current); I think i'll work on it little more during transports.

Today: I want to:

  • look at bugs Oleg created for the sprint, make sure everything is in order (I'm sure it is ! ;) )
  • create a PR with initial spec so that we can discuss on it
  • do asked reviews/needinfo
  • I'll maybe do a simple patch for bug 1146393 for spark now that people dogfood.
  • pick a bug for the sprint

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 1162028 - [Messages][New Gaia Architecture] extract conversation/index.html
    • No progress, just rebased on the top of updated dependency patch (awaiting feedback).
  • bug 1168441 - [Messages][NG] Implement ActivityService
    • Rebased on Steve's bridge PR and made necessary changes + simplified patch. Waiting for bridge.js before asking for review (ready for review, waiting for dependency).
  • 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:

  • Discussed with Steve services we'd like to implement and filed necessary bugs.
    (Julien) do you have the result somewhere?
    (Oleg) On IRC history only :) I can copy and paste it somewhere if you can't recover history
    (Julien) OK I think I'll be able to look at it then - the result will be in the README anyway :)

Today:

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

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