Gaia/SMS/Scrum/1: Difference between revisions
Jump to navigation
Jump to search
Full Query
Full Query
| Line 31: | Line 31: | ||
! Day 5 | ! Day 5 | ||
| 7 | | 7 | ||
|- | |||
! Day 6 | |||
| 5 | |||
|- | |||
! Day 7 | |||
| | |||
|- | |||
! Day 8 | |||
| | |||
|- | |||
! Day 9 | |||
| | |||
|} | |} | ||
Revision as of 07:36, 19 May 2014
List of bugs
http://mzl.la/1fUDBz9: SMS issues handled by the SMS subteam (blocks the sprint bug)
| ID | Assigned to | Summary | Blocking b2g | Whiteboard | Resolution |
|---|---|---|---|---|---|
| 871432 | Steve Chung [:steveck] | [sms][mms] display the received time inside the sms/mms box | --- | [p=2] | FIXED |
| 881469 | Julien Wajsberg [:julienw] | [MMS] Implement Navigation object | --- | [p=3][comms-triaged] | FIXED |
| 963013 | Oleg Zasypkin [:azasypkin] | [Messages][Refresh] Update bubbles style and layout of the thread. | --- | [p=1] | FIXED |
| 980461 | Oleg Zasypkin [:azasypkin] | [MMS] Visual adjustments to attachment download indicator | --- | [p=1] | FIXED |
| 1003384 | Oleg Zasypkin [:azasypkin] | [B2G][1.4][SMS] - The phone type "Other" is not translated in any of the languages when suggested contacts are shown in SMS | 1.4+ | [p=1] LocRun1.4 | FIXED |
| 1008823 | Oleg Zasypkin [:azasypkin] | [Messages] Don't use activity to send message via "Send Message" action from unknown number context menu | --- | [p=1] | FIXED |
| 1010301 | Oleg Zasypkin [:azasypkin] | [B2G]][Messaging App] Spinning icon when sending an sms and mms is not displayed | --- | [p=1][not-part-of-initial-sprint] | FIXED |
7 Total; 7 Open (100%); 0 Resolved (0%); 0 Verified (0%);
Remaining points
| Remaining points | |
|---|---|
| Day 1 | 10 |
| Day 2 | 9 |
| Day 3 | 7 |
| Day 4 | 7 |
| Day 5 | 7 |
| Day 6 | 5 |
| Day 7 | |
| Day 8 | |
| Day 9 |
http://mzl.la/1ggINs4: All SMS issues tracked for this sprint (target milestone)
15 Total; 15 Open (100%); 0 Resolved (0%); 0 Verified (0%);
Sprint planning
The minutes are stored in a separate page.
Daily meetings
Day 2: May, 13th
<azasypkin> hi julienw! <julienw> azasypkin, I failed to send a mail to steve soon enough, and arnau can't join today, so it's only you and me <azasypkin> oh, ok <azasypkin> Soo, my updates from yesterday <azasypkin> 1. Reviewed Doug's patch for 'appendThread' unit tests (landed); <azasypkin> 2. Prepared master patch to get rid of activity from 'Send Message' action (check-in needed); <azasypkin> 3. Prepared small patch to fix activity picker unit tests (landed); <azasypkin> mmm don't remember what else, fixed nits(type=button + copy text) in download indicator patch, but haven't pushed it yet <julienw> okay <julienw> any blocker ? <azasypkin> tried to flash flame device, but it doesn't recognize my SIM <azasypkin> :) <julienw> I didn't try that yet ;) <julienw> and what do you plan for today? <azasypkin> :) seems no any blockers, today will finish with download indicator and probably pick something up from the yesterday list 2.0+ or VR <azasypkin> or do you have smth in mind for me? <julienw> maybe you can take the bug assigned to joan ? <julienw> from yesterday'sl ist <julienw> Arnau answered me that Joan is in PTO <azasypkin> yeah seems entire this week, what bug exactly do you mean? <julienw> https://bugzilla.mozilla.org/show_bug.cgi?id=963013 <firebot> Bug 963013 nor, --, 2.0 S2 (23may), joan.leon, NEW, [Messages][Refresh] Update bubbles style and layout of the thread. <julienw> it may be a good opportunity to use the template mechanism to create the message bubble too <julienw> it may (or may not) smplify the code <azasypkin> mm, ok will see what I can do there. I not sure whether I need Joan's WIP patch <julienw> maybe you can just give a WIP patch that we'll land only with joan's patch? <azasypkin> ok, I need to skim through code, cause I don't really know much about thread layout <julienw> ok :) <julienw> good for you then ! <julienw> my turn then ? <julienw> 1. I updated the Scrum page <julienw> 2. I did a lot of reviews, especially pending ones from last week, and the big composer changes from Joan: bug 963018 <firebot> Bug https://bugzilla.mozilla.org/show_bug.cgi?id=963018 nor, --, 2.0 S2 (23may), joan.leon, NEW, [Messages][Refresh] Update styles of the Composer <julienw> 3. I also reviewed doug's patch that removes the "delete" waiting screen <julienw> and I don't like it :( <julienw> goals for today <julienw> * I don't have much reviews today, so I should be able to do most of them quite early and work on the navigation patch <julienw> * I write down each day the remaining points in our sprint
Day 3: May, 14th
<azasypkin> * Started looking into markup for threads and bubbles to see what I can do for bug 963013; <firebot> Bug https://bugzilla.mozilla.org/show_bug.cgi?id=963013 nor, --, 2.0 S2 (23may), azasypkin, ASSI, [Messages][Refresh] Update bubbles style and layout of the thread. <azasypkin> * Also went through "How to provide the *built-in* data store for messages?" dev-webapi thread updates(https://groups.google.com/forum/#!topic/mozilla.dev.webapi/7fSvp7Dklwk). It looks like we're going to mainly query data from DS (+ probably work with Threads) as too much complexity is hidden in Gecko and it will be difficult and unreasonable to move it to gaia. <azasypkin> There is discussion to make DS is more like shared IndexDB, to support index and rich queries (https://etherpad.mozilla.org/datastore); <julienw> I need to read that discussion too :) <julienw> thanks for sharing <azasypkin> * Played a bit with your Navigation patch. <azasypkin> Today is going to work on bug 963013 and I need to find something else to work on. <azasypkin> that is it :) <julienw> who's next ? <arnau> hey <arnau> I just wanted to follow up on Joans work here <arnau> he is on PTO <arnau> AFAIK he has updated the PR he was working on <arnau> If you could please review it and let me know what's missing I could look at it <julienw> oki <julienw> maybe azasypkin could try to fix the issues we find if he needs something to work on ;) <arnau> only would be good if anyone could help him on the missing javascript part <arnau> cool <julienw> I don't like the idea of joan working while he's in PTO <azasypkin> definitely! <schung> :p <julienw> oki great <schung> my turn? <julienw> azasypkin, I'll talk to you about this later then <julienw> schung, yep please ! <arnau> He is tying to land this patch ASAP <arnau> I know some of us review stuff while on PTO :) <schung> ok. For my side: Fix a regression: Bug 1007441 - Emergency Callback Mode UI not displaying <julienw> because we love this project, but it's not a good life balance ;) <firebot> Bug https://bugzilla.mozilla.org/show_bug.cgi?id=1007441 nor, --, ---, schung, NEW, Emergency Callback Mode UI not displaying <julienw> is it SMS related ? <schung> No, but it's 1.4 blocker and I create this months ago... <julienw> ok ;) <schung> Anyway it's landed in master and waiting QA verify <julienw> oki ! anything else ? <julienw> also what do you plan to do today, and did you have any blockers ? <schung> For the SMS related : Bug 871432 - [sms][mms] display the received time inside the sms/mms box <firebot> Bug https://bugzilla.mozilla.org/show_bug.cgi?id=871432 nor, --, 2.0 S2 (23may), schung, ASSI, [sms][mms] display the received time inside the sms/mms box <schung> I re-submit the patch again for review <julienw> I've seen it ;) <schung> And create another report icon issue for follow up in bug 1010093 <firebot> Bug https://bugzilla.mozilla.org/show_bug.cgi?id=1010093 nor, --, ---, schung, NEW, [Message] Visual refresh for delivery/read report icon in thread view <julienw> btw should this bug be included in our sprint? <julienw> we can see later, one minute left <julienw> my turn :) <julienw> * updated the scrum page to use more general queries (thanks to rik): https://wiki.mozilla.org/Gaia/SMS/Scrum <julienw> * also updated the current's sprint page: https://wiki.mozilla.org/Gaia/SMS/Scrum/1 <julienw> * worked on bug 881469 mainly: tested a lot, small fixes; I'm looking for the TODO that I left and creating bugs for them; I hope to have a review request today for you <firebot> Bug https://bugzilla.mozilla.org/show_bug.cgi?id=881469 nor, --, 2.0 S2 (23may), felash, NEW, [MMS] Implement Navigation object <julienw> * today I have some reviews to do, especially related to joan's patch for visual refresh, and steve's patch too <julienw> * i'll also work on bug 881469 again <julienw> I also want to share gaia try with you: https://tbpl.mozilla.org/?tree=Gaia-Try * arnau est parti (Ping timeout) <julienw> it should eventually replace travis * arnau (Arnau@moz-38025FC2.red-88-1-121.dynamicip.rima-tde.net) a rejoint #gaia-messaging <julienw> one build is started every pull request, like travis <julienw> so it's great
Day 4: May, 15th
<azasypkin> 1. Worked on bug 963013. As it turned out, wrapping header and ul into single container requires more work than I expected :) In scope of this bug I'd like to refactor ThreadUI.getMessageContainer a bit as it's quite big and may go out of control; <firebot> Bug https://bugzilla.mozilla.org/show_bug.cgi?id=963013 nor, --, 2.0 S2 (23may), azasypkin, ASSI, [Messages][Refresh] Update bubbles style and layout of the thread. <azasypkin> 2. Created tiny patch that adds localization note to 'expiry-date-format' that it may not me suitable for all locales; I'm curious whether we're going to use l20n any time soon; <julienw> note that some of getMessageContainer will go away in Steve's bug <julienw> see https://github.com/mozilla-b2g/gaia/pull/16380 <azasypkin> hmm, ok, thanks will check that <azasypkin> 3. Jason Smith filed a regression bug 1010301, it's because of the spinner color change in my "download attachment" styling patch. Spinner is white now, but bubble background is also still white (IFAIK it was going to be green after VR). Asked Victoria on that. I see here two ways: either switch back to blue spinner for now (and make it white in the scope of bug 963013) or wait for bubbles VR (I'm not sure when it's expected to <azasypkin> be landed); <firebot> Bug https://bugzilla.mozilla.org/show_bug.cgi?id=1010301 nor, --, ---, nobody, NEW, [B2G]][Messaging App] Spinning icon when sending an sms and mms is not displayed <azasypkin> 4. In bug 1001439, Stefan Plewako (I guess from Polish localizers team) revealed (well, at least for me :)) that the same date\time format string can be different across locales. So to move '%A, %B %e' format to the shared locale file I'll need to check if that format is the same across all available locales. I'll check that, but it seems I may close this as INVALID if there is nothing to move to shared. <firebot> Bug https://bugzilla.mozilla.org/show_bug.cgi?id=1001439 nor, --, ---, azasypkin, ASSI, [Refactoring] Move '%A, %B %e' date format from app specific files to the shared\locales\date <julienw> oki <azasypkin> Regarding to Steve's patch on refactoring getMessageContainer, maybe it would be better for me to wait for it, I'll check <julienw> you can base your work on it too <julienw> just cherry pick <azasypkin> Today, I'm going to submit patch for bug 963013 for review (probably based on Steve's branch then). And if I have time, will look into other VR bug that we were talking about yesterday. <julienw> great ! <julienw> arnau you're next then ? <arnau> I landed Bug 1004710 <firebot> Bug https://bugzilla.mozilla.org/show_bug.cgi?id=1004710 nor, --, ---, arnau, RESO FIXED, [B2G][Dialer]Creating a new contact and tapping on a text field in the dialer app causes a white bar <julienw> something about SMS ? <arnau> to fix the weird square bugs appearing in copntacts when entering from dialer tabs <arnau> but now I see the dialer delete icon staying on top of contacts <arnau> when you switch tabs <arnau> I'll be looking into it <arnau> Have no more updates from Joan <arnau> I'll talk to him during the day <julienw> oki <julienw> next time it might be handy if we know he'll be in PTO before he is :) <julienw> is it my turn then ? <arnau> sure <arnau> yep, I'm done <julienw> yesterday I mostly worked on bug 881469, I think I'm very close to have a patch ready for review, so hopefully today <firebot> Bug https://bugzilla.mozilla.org/show_bug.cgi?id=881469 nor, --, 2.0 S2 (23may), felash, NEW, [MMS] Implement Navigation object <julienw> today there is an accessibility day here <julienw> so I plan to work on accessibility stuff with some experts we have here <azasypkin> oh, cool <julienw> along with doing my duties <julienw> so that's it :)
Day 5: May, 16th
<julienw> arnau1, schung, azasypkin, it means not more than 3 minutes per person :) <arnau1> from Joan's side <arnau1> I think all he could do is addressed :) <julienw> oki great I'll check it out this morning ! <arnau1> next :) <arnau1> oleg <azasypkin> 1. Worked on bug 1010301 to support different spinner style for incoming and outgoing message bubbles (landed); <firebot> Bug https://bugzilla.mozilla.org/show_bug.cgi?id=1010301 nor, --, 2.0 S2 (23may), azasypkin, RESO FIXED, [B2G]][Messaging App] Spinning icon when sending an sms and mms is not displayed <azasypkin> 2. Worked on bug 963013, prepared PR to wrap date header and message list into single section <firebot> Bug https://bugzilla.mozilla.org/show_bug.cgi?id=963013 nor, --, 2.0 S2 (23may), azasypkin, ASSI, [Messages][Refresh] Update bubbles style and layout of the thread. <azasypkin> + made some minor refactoring while scanning ThreadUI on possible affected pieces. <azasypkin> NI?'ed Joan to preliminarily check whether it what Joan needed to fully resolve this bug. <azasypkin> It's based on the latest Steve's PR, so I'm not sure whether it's fine to ask for review now? <julienw> steve's PR will likely land this morning <azasypkin> nice! <azasypkin> will ask for r? once it's landed and I rebase all the stuff <julienw> do you have anything needed for the day ? <julienw> or do you need more work ? :p <azasypkin> Today I'm going to spend some time (not much :)) to check if it's possible to use flex boxes for recipient_panel+composer+input and to start review of Julien's navigation patch <azasypkin> :) <julienw> ok we can discuss about the recipient panel later if you want <azasypkin> sure <julienw> are you finished ? :) <azasypkin> yep :) next is Steve then <schung> I think Bug 871432 is close to land ;) <firebot> Bug https://bugzilla.mozilla.org/show_bug.cgi?id=871432 nor, --, 2.0 S2 (23may), schung, ASSI, [sms][mms] display the received time inside the sms/mms box <schung> And just look into Bug 963109, but I think navigator object should be first <firebot> Bug https://bugzilla.mozilla.org/show_bug.cgi?id=963109 nor, --, 2.0 S2 (23may), schung, NEW, [Messages] Multi-recipient participants view should show back button instead of cross one <schung> BTW just reviewing the navigator patch(It's still huge even I glanced it 3 month ago :p) <julienw> \o/ <schung> Basic function work fine! I'll take more time for the detail <julienw> bug 963109 is not due for the sprint but there was another bug that you said you'd investigate too <schung> low storage? <julienw> ah yeah <julienw> I couldn't remember :) <schung> Ah sure, I will take a look next week, we should land navigator first <julienw> yep definitely <schung> And some discussion about the delivery/read icon with Omega <schung> That's it <julienw> oki <julienw> so, I worked on the navigation patch :D <julienw> I'm very excited about this because this will let us do a lot of nice things <julienw> I created lots of depending bugs so we can pick some of them in later sprints <julienw> today I plan to do some things I kept aside, most notably reviewing some patches from Doug that should help performance <julienw> and also bugs from contributor that I didn't look much lately <julienw> waiting for your review :p <julienw> otherwise the sprint is looking good, we'll be at 5 remaining points today, out of 10 <schung> you could redirect to us if you have no time <julienw> yep that's right I don't do this enough :)