Confirmed users
383
edits
Gaia/SMS/Scrum/FxOS-S8: Difference between revisions
m
→Day 10: 5th October
Steveck324 (talk | contribs) |
|||
| Line 404: | Line 404: | ||
=== Day 10: 5th October === | === Day 10: 5th October === | ||
====Steve==== | |||
* {{Bug|1179628}} - [Messages][NG] Lay out Settings service structure | |||
** Background work, no progress. | |||
* {{Bug|1201016}} - [Messages][NG] Migrate the current Message manager event handling to NGA. | |||
** No progress. | |||
* {{Bug|1207094}} - [Messages] Disable appropriate controls when in low storage condition | |||
** Create a WIP for early feedback, will list some missing part for UX on bug and some drawback in the code on github. Waiting for early feedback | |||
* {{Bug|1206678}} - Outgoing MMS contains subject that was typed by user, but then hidden | |||
** Landed. | |||
* Rich text copy paste: Boris will disable it for 2.5 | |||
*: (Julien) OK, good to know; we can plan ahead our work to support it post-2.5 :) | |||
*: (Steve) I think they don't have any idea about the proper handle inside message, will keep in touch. | |||
*:(Julien) Email has the same issue; we need an event to handle it, instead of Gecko handling it automatically... | |||
*:(Steve) It seems the only way if we want to insert the image as attachment. Maybe we can simply have a element attribute or css property to limit the text past only in message input for near future. | |||
*:(Julien) yeah, but it would be really cool to support image paste :D | |||
*:(Steve) Yeah I understand ;) | |||
* Reviewing the systemMessage handler, it's not finished yet but it should be in the right direction. And we want to rewrite message receiving part for long time ago so it's good to see the changes. | |||
Today: | |||
* Low storage work. | |||
* Give some feedback for Oleg's patch about the system message. | |||
* Working on some blockers if they have any progress. | |||
====Julien==== | |||
Sorry, not much happened related to SMS for me :) | |||
In case you don't know, the "low storage" feature got a "no go", so it's no more 2.5... We can still go on and finish it behind a flag though. | |||
:(Oleg) I'm curious what happened? :) | |||
*:(Julien) I think they fear about regressions. | |||
*:(Steve) The decision making process is horrible in 2.5.... | |||
*:(Julien) EPM and some developers (incl me) started looking at the feature in August but there has been a lot of inertia from some teams, likely because they were also busy with other stuff. We don't plan ahead enough with "big blocks of features", somewhat like we did for Milestone 1 (but that we didn't do since then BTW :) ) | |||
*: (Steve) So how about the ongoing low storage patches, like finish them and disabled or simply pending? | |||
*: (Julien) for the dialog patch, it's easy because nobody uses it, so I think i'll just go ahead and merge; for the "input disabling", I think it should be disabled with a boolean in our Settings object. What do you think ? | |||
*: (Steve) I do have a flag(it's in device storage client instead of settings) in the patch, so it's fine for me. | |||
*:(Julien) OK ! (I think we should have it in Settings though :p) | |||
Got an idea today about how we could use "async" scripts, I'd like to experiment... maybe next "hacking day" ;) | |||
:(Oleg) Ah, I've just realized we didn't have hack day this month? Or I just missed? | |||
:(Julien) nope, nobody scheduled it (and I still haven't sent the result of last one -- will try to do it today :/) | |||
Today: | |||
I want to: | |||
* read bugmail | |||
* handle review/feedback/NI queue | |||
* continue work on the low storage condition | |||
====Oleg==== | |||
* I'll be on PTO on Friday (moved my b-day Thursday PTO to Friday :)) | |||
* {{Bug|1198266}} - [Messages] Use ConversationService in the application | |||
** No progress (awaiting feedback). | |||
* {{Bug|1208532}} - [Messages][Refactoring] Extract SystemMessageHandler from ActivityHandler | |||
** Fixed all affected unit tests, especially refactored sms-received and notification ones, started to work on few integration tests for the sms-received system message just to have something for the start (one for sms and one for class-0 message as we don't have any for class-0 messages) (awaiting feedback, in progress). | |||
Other: | |||
* {{Bug|1206844}} - Adding Firefox Emoji font support | |||
** Worked with Pavel to finish the patch, proposed simpler way to style "sending" state messages that contain emojis, waiting for Pavel's reply to Tim (honestly I'm not sure how it should be done in ideal case). Do you guys have any ideas on how it should be handled? Should gecko manage it somehow as Tim suggests or ... ? | |||
**: (Julien) if you add a feedback or needinfo on the bug, I'll try to look at it; otherwise I'll forget :p | |||
**: (Oleg) Sure, will ni? :) | |||
* {{Bug|1206678}} - Outgoing MMS contains subject that was typed by user, but then hidden | |||
** Reviewed PR. | |||
* {{Bug|1148719}} - [Message]The recipient won't be saved in draft if we delete all message in this conversition while editing new message. | |||
** Explained old idea to UX guys, got reply today - need to read through it. | |||
Today: | |||
* Will handle review/feedback/need-info requests; | |||
* Will work on review comments and assigned bugs. | |||
== Demos == | == Demos == | ||
== Retrospective == | == Retrospective == | ||