Confirmed users
383
edits
Gaia/SMS/Scrum/FxOS-S9: Difference between revisions
Jump to navigation
Jump to search
m
→Day 7: 15th October
| Line 314: | Line 314: | ||
=== Day 7: 15th October === | === Day 7: 15th October === | ||
====Steve==== | |||
* {{Bug|1211395}} - Performance regression in Message | |||
** Apply the style lazyloader and hide the edit modeform for shaving 100ms. Ceate a patch for early feedback and please let me know if the patch does improve somehow. | |||
* {{Bug|1192263}} - [Messages] We load Inbox before going to the notification conversation when app is run via notification click | |||
** Add a unit test, but wondering where to add the integration test for it. It looks like we can resue the notification test and add assertion in openNotification. Oleg, do you think it make sense or I should assert in somewhere else, or even write another test for this? | |||
**: (Oleg) IMO notification test sounds good, the suite name sounds appropriate "'Run application via notification'" :) | |||
**: (Julien) not sure how to do a solid integration test for this. I'll read your proposal :) | |||
**: (Oleg) Heh, yeah, I was also wondering if it's not too hard :) | |||
**: (Steve) TBH I'm not sure how to write a proper integration test for it :p | |||
**: (Julien) if we don't have a good idea, I'd be happy enough with only a unit test. | |||
**:(Steve) I might still need to touch the integration test because this changes seems break 3rd test in notifiction test, still finding the root cause. | |||
**:(Julien) ok ! | |||
Today: | |||
* Polishing the edit form display for {{Bug|1211395}} and wait for feedbacks. | |||
* Working on some blockers if they have any progress. | |||
====Julien==== | |||
Not a very productive day, I felt not so good yesterday. | |||
* continued work on {{Bug|1206727}}. I'm considering having a follow-up to extract the conversation rendering logic to an external object -- but not here because I want this to be risk-less. | |||
* fixed {{Bug|1214623}} to prevent test files from being packaged. I thought this could have a performance impact, but nada :) - I've learnt new Spanish (?) "nada" word :) | |||
* finished handling my old bugmail ! | |||
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. | |||
** Handled review comments and asked for review (in review). | |||
* {{Bug|1197231}} - Increase in SMS app memory consumption | |||
** Tried to analyze about:memory reports, a lot of stuff is unclear - asked for a help from platform guys; Comparing it with v2.2, trying to spot something obvious (in progres); | |||
** Tried small tweaks like removing event listeners from lazy loaded scripts (we have about 160 active event listeners in inbox per about:memory report) - nothing significant; | |||
** Compiling build with DMD enabled to see if it can give more details (https://developer.mozilla.org/en-US/docs/Mozilla/Performance/DMD#B2G_2). | |||
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|1133711}} - Remove dev_apps folder (in background) | |||
** No progress so far, waiting for ni? I've set (waiting for ni?) | |||
Today: | |||
* Will handle review/feedback/need-info requests; | |||
* Will work on review comments and assigned bugs. | |||
=== Day 8: 16th October === | === Day 8: 16th October === | ||
=== Day 9: 19th October === | === Day 9: 19th October === | ||
== Demos == | == Demos == | ||
== Retrospective == | == Retrospective == | ||