Confirmed users
383
edits
Gaia/SMS/Scrum/FxOS-S9: Difference between revisions
Jump to navigation
Jump to search
m
→Day 8: 16th October
| Line 361: | Line 361: | ||
=== Day 8: 16th October === | === Day 8: 16th October === | ||
====Steve==== | |||
* {{Bug|1211395}} - Performance regression in Message | |||
** Got f+ and replied some thought. Commit another update with small changes about the lazyload more styles and element hiding. Will work on unit tests. | |||
* {{Bug|1192263}} - [Messages] We load Inbox before going to the notification conversation when app is run via notification click | |||
** Figured out the problem we have with current patch. Replied some thought as well on bugzilla. | |||
Today: | |||
* Will update the unit test part for performce patch. | |||
* Working on some blockers if they have any progress. | |||
* Review and reply ni. | |||
====Julien==== | |||
I decided to work on some blockers yesterday. | |||
* worked on a blocker in Contacts ({{Bug|1213553}}); especially spent some time on writing integration tests for pages opened by window.open but it didn't work ;) Will likely be useful for us later ! | |||
* worked on {{Bug|1215312}} that is a follow-up for the late arrival {{Bug|1037620}}. (In review; steve I would appreciate if you look at it today, I'd like it in the next OTA if it's r+ :) ). | |||
* continued some little work on {{Bug|1206727}}. | |||
* had a look at {{Bug|1192263}} (start the app from notification system message); sorry my proposed solution is a lot of work :( | |||
** :(Steve) I left some comments and maybe we can only check visibility and close the window first. Not sure if we really need taskRunner for it. | |||
* r+ed {{Bug|1213199}} about the draft notice being shown at wrong moments | |||
* reviewed {{Bug|1210731}} refactoring the RTL code in our CSS | |||
* feedback+ed {{Bug|1197231}} about the memory consumption. More idea I got from Nicholas comment: our lazy loaded JS files are not minified... so I started a change in the build to do this) | |||
**:(Oleg) But it was the same on v2.2, no (note I've used the same Messages app v2.2 for both reports)? | |||
**:(Steve) no we didn't minified the lazyload scripts on 2.2 either. But we lazyload more scripts in master :p | |||
**:(Julien) ok :) | |||
**:(Julien) yeah, but if we have more scripts, this could still help us go down :) Okay :) | |||
**:(Julien) same thing than about Steve's patch to lazy load CSS: we had them before too ;) but now that it's better split, we can do this. | |||
* feedback+ed {{Bug|1211395}} about the launch time regression | |||
Others: | |||
* I now have the bugzilla right to add the editbugs right to people :) | |||
* filed {{Bug|1215047}} to be nicer with attachment names with letters that are not in the 7-bit ASCII space | |||
Today: | |||
I want to: | |||
* handle my review queue | |||
* try to wrap up {{Bug|1206727}} to ask for a first feedback | |||
* maybe try to fix some blockers in other apps. | |||
* land some r+ bugs I have somewhere | |||
====Oleg==== | |||
* {{Bug|1198266}} - [Messages] Use ConversationService in the application | |||
** No progress (awaiting feedback). | |||
* {{Bug|1213199}} - [Message] Wrong suggestions 'Message saved as draft' pop up after you select delete the MMS. | |||
** Got r+, waiting for Treeherder (single set view doesn't work due to "{{Bug|1215450}} - Treeherder single result set view is stuck "waiting for a push with revision xxx"") (almost landed). | |||
* {{Bug|1197231}} - Increase in SMS app memory consumption | |||
** More analysis, DMD didn't give any info I can really interpret; | |||
** But I was able to figure out the reason why Gaia v2.2 didn't work on Gecko v2.5 for me - it's new web components permission, I've added it and running raptor - I want to have as concrete info as possible about Gecko regression, will file a bug with raptor results and memory reports. Do you know guys who will be the best Gecko person to ni? (in progress); | |||
**:(Julien) IMO Nicholas is really the best here. He worked a lot on reducing memory in Gecko. | |||
**:(Oleg) Okay, so will ni? him with as much info as I can get :) | |||
** Got reply from Nicholas. | |||
Other: | |||
* {{Bug|1209419}} - [Messages][Tests] Add basic integration tests for the Conversation view Edit mode | |||
** Again, left feedback and guided on the next steps (in progress). | |||
* {{Bug|1211395}} - Performance regression in Message | |||
** Left feedback for Steve's patch, mostly tested on device and quickly skimmed through the patch, looks good :) | |||
* {{Bug|1133711}} - Remove dev_apps folder (in background) | |||
** No progress so far, waiting for ni? I've set (waiting for ni?). Likely need to ping guys directly on IRC :/ | |||
Today: | |||
* Will handle review/feedback/need-info requests; | |||
* Will work on review comments and assigned bugs. | |||
=== Day 9: 19th October === | === Day 9: 19th October === | ||
== Demos == | == Demos == | ||
== Retrospective == | == Retrospective == | ||