Confirmed users
820
edits
Gaia/SMS/Scrum/3: Difference between revisions
Jump to navigation
Jump to search
→Day 2: 11th June
| Line 108: | Line 108: | ||
Today: will work with Pavel on subject handling and CSP compliance bug. | Today: will work with Pavel on subject handling and CSP compliance bug. | ||
===Day 3: 12th June=== | |||
====Steve==== | |||
* Bug 1016897 - [Flame][v1.4][Message]Tap new message icon, it will automatically return SMS which is just saved as draft .And if then tap Back key, this contact will have 2 SMS items in main message view. | |||
** Patch landed | |||
* Bug 994553 - [Sora][Message][Contacts]Contacts's head portrait do not display in Message | |||
** Patch given, it revoke the url when element deleted. Hope it works | |||
* Bug 1021788 - [Messages] Can't click on the suggestion list when there are a lot of recipients in the recipients panel | |||
** Patch given | |||
* bug reviewing: | |||
** bug 1013296: Send button VR. Got dependency with subject VR and DSDS VR. No progress, wait for other blocker | |||
** bug 963043: No update | |||
** bug 1015867: Since we know the root cause and Bug 1021788 is tracking it, we can keep the review moving forward and land it today. | |||
Today: | |||
* Wait for review | |||
* Take other blocker or triage other bugs if nothing really blocked | |||
* Review bug 1015867 | |||
* Take a look for haida plaining | |||
====Julien==== | |||
* mostly reviews: | |||
** bug 974867 for email suggestion and bug 997547 for send mms by email activity; I requested to put this behind a flag while it's not finished, because landing one of these patches only will break things | |||
** other minor things | |||
* add a look on my remaining bug for the sprint bug 1015867 for the composer length | |||
** we agreed with Steve that the remaining issue was actually already filed in a separate bug and unrelated | |||
* so should land today and unblock other work \o/ | |||
* add a look to some other blockers, requested information: | |||
** bug 1020653: no vibration when receiving a message in some situation. The regression window doesn't help much, so I'll investigate more today | |||
** bug 1022755 reported by Etienne, navigation issue and I saw it myself, but it's not really 100% reproducible for me yet, I'll ask Etienne today | |||
* still some bug triage | |||
Other: | |||
* updated the Gaia/SMS page: https://wiki.mozilla.org/Gaia/SMS | |||
Today: | |||
* some more left reviews (my goal is to have one less review left when I leave work :) ) | |||
* want to at last look to the datastore etherpad | |||
====Oleg==== | |||
* Bug 1008127 - [Messages][Refresh] Subject handling in the Composer | |||
** Pinged Pavel once again as it looks like PR is rebased incorrectly + still waiting ni? from Vicky. | |||
* Bug 1012663 - [SMS] Remove inline style for CSP compliance | |||
** Refactoring of attachment.js render method took a bit more time than I expected :) Still working on it + updating related unit tests. Should finish with it today and send for review. | |||
* Other: My internet is finally fixed! :) | |||
Today: will be finishing with patch for bug 1012663 and, yes, will take a look at haida etherpad, that sounds quite interesting. Also was planning to look at regression as my commits are in regression window push log Bug 1018035 - [Flame][v1.4][SMS]No cursor display when tap the message edit box. | |||
:Julien have you already looked into it? | |||
:(Julien) yep, couldn't reproduce yet but I didn't use the correct STR. I can continue investigating on my side today and if you're free tomorrow you can possibly steal it :) | |||
:(Oleg) Ok! | |||
== Demos == | == Demos == | ||
== Retrospective == | == Retrospective == | ||