Confirmed users
820
edits
(Day 11: 10th June) |
(Day 11: 10th June) |
||
Line 554: | Line 554: | ||
=== Day 11: 10th June === | === Day 11: 10th June === | ||
====Steve==== | |||
Steve | |||
* Off-site training today | * Off-site training today | ||
* {{Bug|1169541}} - [Messages] [NGA] Refine the thread query and move draft query/getThreads API into conversation service / messaging service | |||
* Bug 1169541 - [Messages] [NGA] Refine the thread query and move draft query/getThreads API into conversation service / messaging service | |||
** No progress | ** No progress | ||
* {{Bug|1169573}} - [Messages][NG] Lay out Messaging service structure | |||
* Bug 1169573 - [Messages][NG] Lay out Messaging service structure | |||
** pended by mozMobileMessageShim | ** pended by mozMobileMessageShim | ||
* {{Bug|1171865}} - [Messages][NG] Layout mozMobileMessageShim structure | |||
* Bug 1171865 - [Messages][NG] Layout mozMobileMessageShim structure | |||
** Update the patch based on feedback. | ** Update the patch based on feedback. | ||
Today: | Today: | ||
* Create a complete API shim patch for review | * Create a complete API shim patch for review | ||
Line 585: | Line 572: | ||
** file loading in Template.js | ** file loading in Template.js | ||
Julien | ====Julien==== | ||
* finished review in {{Bug|1162040}} (accessibility improvement) | |||
* finished review in | * reviewed Oleg's work in {{Bug|1168441}} (Activity Shim) | ||
* reviewed Oleg's work in Bug 1168441 (Activity Shim) | * reviewed Gandalf's migation to l10n v3 in {{Bug|1171206}} (huge work !) | ||
* reviewed Gandalf's migation to l10n v3 in | * wrote a small patch for {{Bug|1146393}} so that images taken in Aries can have a thumbnail | ||
* wrote a small patch for | * thought there was an issue with thread loading ({{Bug|1172896}}) but this is only how it looks like -- I'll see if I can do something quick about it | ||
* thought there was an issue with thread loading ( | * filed meta bug for NGA: {{Bug|1172906}} | ||
* filed meta bug for NGA: | |||
Others: | Others: | ||
* fixed an issue in the reference workload script ( | * fixed an issue in the reference workload script ({{Bug|1172864}}) so that the master script work with older versions + better clean up when we run it on top of an existing DB | ||
Today: | Today: | ||
I want to: | I want to: | ||
* do asked reviews/feedbacks asap; goal is to load the conversation split today ! ;) | * do asked reviews/feedbacks asap; goal is to load the conversation split today ! ;) | ||
* pick another bug for the sprint | * pick another bug for the sprint | ||
If all this moves forward well, I could: | If all this moves forward well, I could: | ||
* continue the prototype caching the thread list to a single db (including contacts/drafts/etc). | * 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 ( | * I'd like to work on the deduplication logic for network alerts, it seems important for users ({{Bug|1067938}}) | ||
====Oleg==== | |||
It was "struggling with issues" day :) | It was "struggling with issues" day :) | ||
* {{Bug|1162028}} - [Messages][New Gaia Architecture] extract conversation/index.html | |||
* Bug 1162028 - [Messages][New Gaia Architecture] extract conversation/index.html | |||
** No updates, will rebase both PRs on the latest master and incorporate aria-label changes that landed in the meantime (awaiting feedback). | ** No updates, will rebase both PRs on the latest master and incorporate aria-label changes that landed in the meantime (awaiting feedback). | ||
* {{Bug|1168441}} - [Messages][NG] Implement ActivityService | |||
* Bug 1168441 - [Messages][NG] Implement ActivityService | |||
** Fixed review comments, renamed files to be in line with what we agreed to do for view controllers, got r+ and landed, yay! (landed). | ** Fixed review comments, renamed files to be in line with what we agreed to do for view controllers, got r+ and landed, yay! (landed). | ||
Other: | Other: | ||
* Helped Johan to find typo in his integration test PR -https://github.com/mozilla-b2g/gaia/pull/30236#issuecomment-110438557; | * Helped Johan to find typo in his integration test PR -https://github.com/mozilla-b2g/gaia/pull/30236#issuecomment-110438557; | ||
* Tried to setup forwarding in SW (views/new_message/index.html to views/conversation/index.html), but it didn't work and crashed the browser - prepared test case and filed "Bug 1172884 - Synthesized permanent redirected response crashes browser"; | * Tried to setup forwarding in SW (views/new_message/index.html to views/conversation/index.html), but it didn't work and crashed the browser - prepared test case and filed "{{Bug|1172884}} - Synthesized permanent redirected response crashes browser"; | ||
* Tried to figure out with Fernando why Message app with ServiceWorker registration doesn't work in b2g using app:// protocol. Created simplified test app to make easier to spot the issue for Fernando: https://github.com/azasypkin/sw-tests/tree/master/fxos-app. | * Tried to figure out with Fernando why Message app with ServiceWorker registration doesn't work in b2g using app:// protocol. Created simplified test app to make easier to spot the issue for Fernando: https://github.com/azasypkin/sw-tests/tree/master/fxos-app. | ||
* Discussed our threads/bridge experience with Wilson (https://github.com/gaia-components/threads/issues/40 that will help us to remove setTimeout in ActivityShim) and added comments into simplified threads proposal. | * Discussed our threads/bridge experience with Wilson (https://github.com/gaia-components/threads/issues/40 that will help us to remove setTimeout in ActivityShim) and added comments into simplified threads proposal. | ||
Line 627: | Line 606: | ||
*:(Oleg) I think we can leverage Babel somehow. Yeah, arrows are sooooo goood :) | *:(Oleg) I think we can leverage Babel somehow. Yeah, arrows are sooooo goood :) | ||
*: (Julien) yeah maybe ! | *: (Julien) yeah maybe ! | ||
Today: | Today: | ||
* Will handle review/feedback/need-info requests; | * Will handle review/feedback/need-info requests; |