Gaia/SMS/Scrum/FxOS-S4

From New wiki
< Gaia | SMS | Scrum
Jump to navigation Jump to search

List of bugs

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

Bugzilla link

<bugzilla> { 

 "blocks": "1188768",
 "include_fields": "id, assigned_to, summary, cf_blocking_b2g, cf_feature-b2g, whiteboard, resolution"

} </bugzilla>

Remaining points and burndown chart

google chart api url for Sprint FxOS-S4

Burndown chart
Remaining points
Start 7
Day 2 7
Day 3 7
Day 4 7
Day 5
Day 6
Day 7
Day 8
Day 9
Day 10
End 4


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

<bugzilla> { 

 "whiteboard": "sms-sprint-FxOS-S4",
 "include_fields": "id, assigned_to, summary, cf_blocking_b2g, cf_feature-b2g, whiteboard, resolution"

} </bugzilla>

All SMS issues tracked for this sprint (target milestone)

Bugzilla link

<bugzilla> { 

 "component": "Gaia::Sms",
 "target_milestone": "FxOS-S4 (07Aug)",
 "include_fields": "id, assigned_to, summary, cf_blocking_b2g, cf_feature_b2g, resolution"

} </bugzilla>

Sprint planning

Minutes are on a separate page.

Daily meetings

Day 2: 29th July

Steve

  • bug 1184865 - [Messages][NG] Replace some methods in MessageManager with messaging service in conversation view.
    • Polish the patch and adding the tests/desktop mock
  • bug 1179628 - [Messages][NG] Lay out Settings service structure
    • Start mozMobileConnections.
  • bug 1180592 - [Messages][NG] mozMobileConnections shim Implementation
    • Layout for mozMobileConnections shim first.
  • bug 1167144 - [Messages] Reduce the use of Threads.active and Threads.currentId in conversation view
    • Update for the latest suggestions.

Today:

  • Layout for mozConnections shim
  • Figure out the problem in messaging service and profiling.
  • Review Julien's navigation patch

Julien

  • Hackday event

Oleg

  • Hackday event

Day 3: 30th July

Day 4: 31st July

Steve

  • bug 1184865 - [Messages][NG] Replace some methods in MessageManager with messaging service in conversation view.
    • Found one root cause that make it return no response. We must clone the message then return the cloned message from shim. Have no idea why there is no data clone error in the log.
  • bug 1179628 - [Messages][NG] Lay out Settings service structure
    • Start mozMobileConnections.
  • bug 1180592 - [Messages][NG] mozMobileConnections shim Implementation
    • Layout for mozMobileConnections shim first. No progress yet.
  • bug 1167144 - [Messages] Reduce the use of Threads.active and Threads.currentId in conversation view
    • In review(Might have some conflict with navigation patch, so hold it first).
  • Reviewing the navigation patch. Still in progress but left some questions first

Today:

  • Figure out the problem in messaging service and profiling.
  • Layout for mozConnections shim
  • Review Julien's navigation patch

Julien

  • still worked on navigation (bug 1162030):
    • not much progress on last day I worked (tuesday ;) ) Just fixed some early comments and pushed a new commit.
  • finished reading most of my bug mail
  • handled some reviews
    • gave feedback on bug 1184865 that starts using services for MessagingManager
    • gave feedback on bug 1169576 that implements the service for getting the threads

Today: I want to:

  • do reviews
  • handle comments for the navigation patch

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 1155534 - [Messages][NG] Extract NewMessage view from Conversation view
    • No updates (in background).
  • bug 1176976 - [Messages][Drafts] Remove the draft saving/replacing action menu
    • No updates, will reply to some comments (in review).
  • bug 1169576 - [Messages][NG] Implement Conversation service: method for streaming joined threads and drafts list
    • Replied to questions, working on comments and tests (in progress).

Other:

  • Spend 50% of my day reviewing "bug 1162030 - [Messages][NG] Figure out how navigation works" but still not finished, nothing serious just carefully reading to better understand, only startup.js and testing on device left :).

Today:

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

Day 5-9: 3rd to 7th August

Meetup in Paris. See https://groups.google.com/d/msg/mozilla.dev.gaia/driAONuqNT0/dId0_NefDgAJ for a summary.

Day 10: 10th August

No update.

Day 11: 11th August

Steve

  • bug 1184865 - [Messages][NG] Replace some methods in MessageManager with messaging service in conversation view.
    • Requesting reviewing. Polishing per feedbacks.
  • bug 1179628 - [Messages][NG] Lay out Settings service structure
    • Start mozMobileConnections.
  • bug 1180592 - [Messages][NG] mozMobileConnections shim Implementation
    • Layout for mozMobileConnections shim first. No progress yet.
  • bug 1167144 - [Messages] Reduce the use of Threads.active and Threads.currentId in conversation view
    • In review(Might have some conflict with navigation patch, so hold it first).
  • Have some discussion with UX about removing the unnecessary dynamic contact image and possibility of removing the checkboxes from list template.
    (Julien) dynamic contact image: the one that is supposed to get the first letter of the contact name ? And they want to remove the checkbox and instead show the selection with for example a background color ?
    (Steve) Just think that the use of nth-child in group picture css seems not necessary and it might bring performance concern. For the checkbox selection, I did propose the replacing the cb with background color to Morpheus , but he didn't agree with this idea for UX consistency. But maybe we can still try to inset the cb only in selection mode(Oleg's idea).
    (Julien) to replace the nth-child, we can simply use classes generated in JS if we still want it.
    (Steve) I just think that we can use static image for group contact. Don't feeling the background differentiating could bring any benefit to user.
    (Julien) regarding the checkbox: if we add it when we enter edit mode, this produces a reflow, and I think we want to avoid it. Regarding the different background, this is only design :D

Today:

  • Revise the messaging client patch.
  • Figure out the problem that blocking the scroll event while thread DB cursor continuous traversing.
  • Layout for mozConnections shim

Julien

  • worked on 2 follow-ups for the navigation patch:
    • bug 1190276: activity "new" does not behave correctly in some situations (in review)
    • bug 1190275: images attachments are not visible in some situations (in review)
  • also created more follow-ups for the patch
  • did a lot of reviews
    • bug 1192599: small patch to remove "innerHTML" translations
    • bug 1167144: reduce use of Threads.active
    • bug 1176976: remove draft dialogs -- was not possible to review because the rebase was quite big :/
    • bug 1184865: move retrieveMMS to services
    • bug 1136147: mentored bug, helped the user to rr to reproduce it by making a small patch to desktop-mock

Today: I want to:

  • do more reviews
  • work on the system messages
  • possibly work on 1 simple follow-up to the navigation patch

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 1155534 - [Messages][NG] Extract NewMessage view from Conversation view
    • No updates (in background).
  • bug 1176976 - [Messages][Drafts] Remove the draft saving/replacing action menu
    • Will rebase on the latest master and ask review (in progress).
  • bug 1169576 - [Messages][NG] Implement Conversation service: method for streaming joined threads and drafts list
    • Was working on this bug during WW, implemented SharedWorker-connection-per-app-instance to support several Messages instances at the same time (we needed this work because we use BroadcastChannel to for SharedWorker->IFrame communication and brings problems) (in progress);
    • Started writing unit tests for this patch.

Other:

  • Reviewed "bug 1162030 - [Messages][NG] Figure out how navigation works".

Today:

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

Demos

Retrospective

Retrieved from "https://mozilla.mediawiki.site/w/index.php?title=Gaia/SMS/Scrum/FxOS-S4&oldid=1089610"