Confirmed users
383
edits
Gaia/SMS/Scrum/2.1S4: Difference between revisions
Jump to navigation
Jump to search
→Day 2: 3rd September
Steveck324 (talk | contribs) |
|||
| Line 79: | Line 79: | ||
== Daily meetings == | == Daily meetings == | ||
=== Day 2: 3rd September === | === Day 2: 3rd September === | ||
====Oleg==== | |||
* {{Bug|1061150}} - [Messages][Refresh] Change outgoing message background to #fff | |||
** Prepared patch for background and bubble box-shadow, got r+, landed (landed). | |||
* {{Bug|1048717}} - [Messages][Refresh] Display call button at the header of single recipient thread | |||
** Started to work on this (in progress); | |||
** Noticed that if we have invalid number (like sender equals to string, in my case operator sends messages with sender === 'FehlerInfo'), we can't easily validate number, so we display call option, but if user tap on it - nothing will happen as telephony activity is just rejected. Talked to Rik about it, probably solution for that will be "{{Bug|1032273}} - [v2.1] Bug: Tapping "Call" launches Phone but does not dial" where we call directly from SMS + probably some shared with Dialer\Emergence Call error handling and number validation code. | |||
* {{Bug|1061215}} - [Messages][Refactoring] Improve handling of remaining chars counter | |||
** Follow-up for subject handling patch, have a WIP (in progress). | |||
* {{Bug|1061682}} - [Messages] Header shows wrong icons for back\close actions | |||
** Noticed that regression caused by the latest gaia-header patch (checkin-needed, v2.1?) | |||
* {{Bug|1048845}} - [Messages] Compose Panel refresh (subject behavior change) | |||
** Landed (landed). | |||
* {{Bug|1061710}} - [Flame][SMS] Unable to send sms | |||
** Was investigating, it's hard to reproduce but still possible. There is the same problem I saw some time ago with CMAS - settings.createLock.get doesn't return, it just hangs somewhere, for that reason MultiSimAction button doesn't work - neither executes send nor shows correct sim number. | |||
** I believe there are other apps affected by this, will look for similar bugs, otherwise will try to find good STR. | |||
** Just noticed Steve's reference to might have caused this "{{Bug|1061510}} - [B2G] JavaScript Error: "ReadOnlyError: A mutation operation was attempted in a READ_ONLY transaction.". | |||
Other: | |||
* Reviewed CMAS duplicated notification bug, r+'d | |||
* Added Demos to wiki page. | |||
Today: | |||
* Will handle review comments for the patches that currently in review; | |||
* Will deal with review\feedback requests; | |||
=== Day 3: 4st September === | === Day 3: 4st September === | ||