Confirmed users
820
edits
Gaia/SMS/Scrum/1: Difference between revisions
→Daily meetings
| Line 135: | Line 135: | ||
<julienw> my turn :) | <julienw> my turn :) | ||
<julienw> * updated the scrum page to use more general queries (thanks to rik): https://wiki.mozilla.org/Gaia/SMS/Scrum | <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> * 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 | <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 | ||
| Line 148: | Line 146: | ||
<julienw> one build is started every pull request, like travis | <julienw> one build is started every pull request, like travis | ||
<julienw> so it's great | <julienw> so it's great | ||
</pre> | |||
=== Day 4: May, 15th === | |||
<pre> | |||
<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 :) | |||
</pre> | </pre> | ||