Gaia/SMS/Scrum/2.1S4
Jump to navigation
Jump to search
Full Query
Full Query
Full Query
List of bugs
SMS issues handled by the SMS subteam (blocks the sprint bug 1061491)
| ID | Assigned to | Summary | Blocking b2g | Feature-b2g | Whiteboard | Resolution |
|---|---|---|---|---|---|---|
| 983172 | Steve Chung [:steveck] | Parsing jpeg header information for downsampling the image for thumbnail | --- | No cf_feature-b2g | [sms-most-wanted][p=2] | FIXED |
| 1048717 | Oleg Zasypkin [:azasypkin] | [Messages][Refresh] Display call button at the header of single recipient thread | --- | No cf_feature-b2g | [p=1] | FIXED |
| 1053952 | Oleg Zasypkin [:azasypkin] | [Messages][Refactoring] Delete all messages at once when deleting threads | --- | No cf_feature-b2g | [p=1] | FIXED |
| 1059233 | Steve Chung [:steveck] | [SMS] User cannot view the cropped image in SMS application and ActivityCanceled error was shown when try to view | --- | No cf_feature-b2g | [LibGLA,TD91659,QE1, B][p=1] | DUPLICATE |
4 Total; 4 Open (100%); 0 Resolved (0%); 0 Verified (0%);
Remaining points and burndown chart
google chart api url for Sprint 2.1S4
| Remaining points | |
|---|---|
| Start | |
| Day 2 | |
| Day 3 | |
| Day 4 | |
| Day 5 | |
| Day 6 | |
| Day 7 | |
| Day 8 | |
| Day 9 | |
| Day 10 | |
| End |
SMS issues handled by the SMS subteam outside of the sprint (contains whiteboard "sms-sprint-2.1S4")
| ID | Assigned to | Summary | Blocking b2g | Feature-b2g | Whiteboard | Resolution |
|---|---|---|---|---|---|---|
| 1053964 | Oleg Zasypkin [:azasypkin] | [Messages][Tests] Add marionette JS test case to verify close\back button behaviour when app is in activity mode | --- | No cf_feature-b2g | [sms-most-wanted][sms-sprint-2.1S4] | FIXED |
| 1063970 | Oleg Zasypkin [:azasypkin] | [Messages][Refresh] Update styles of phone and url links located inside messages | --- | No cf_feature-b2g | [sms-sprint-2.1S4] | FIXED |
2 Total; 2 Open (100%); 0 Resolved (0%); 0 Verified (0%);
All SMS issues tracked for this sprint (target milestone)
17 Total; 17 Open (100%); 0 Resolved (0%); 0 Verified (0%);
Sprint planning
Minutes are on a separate page.
Daily meetings
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
Steve
- bug 1060727 - [CMAS] Duplicated notifications in system tray when dismiss the attention screen by tapping on home button
- Landed on master and create a follow up bug 1062104 for future system changes
- bug 1048841 - [Messages] investigate performance hit from IndexedDB at startup (eg: Contact lookup)
- Waiting for the QC's reply. If they takes the gecko patch, we might need to provide the corresponding gaia patch to fix the profiling result
- bug 983172 - Parsing jpeg header information for downsampling the image for thumbnail
- It's time to finish this patch anyway. Will also apply the square thumbnail visual changes as well.
- (Oleg) I also see we have one more bug with incorrect attachment alignment
- (Steve) Yes, I saw it too. Do you think we should separate them and fix?
- (Oleg) I guess we should (if we need to uplift this to v2.0), I see that reporter says it happens for v2.0 too, so I can take a look what broke this and then we can decide :)
- (Steve) Hmm ,it would be reasonable to separate them if it's 2.0 blocker...
- (Oleg) Btw, do you plan to uplift patch for bug 983172 to v2.1?
- (Steve) I think we should. Memory leak is an irresistible reason ;) Yep!
- (Oleg) I also think so! So if we uplift it and fix for wrong alignment is one-liner or so, we can bundle it with your patch for v2.1. So I'll figure out what the reason for this alignment issue
- It's time to finish this patch anyway. Will also apply the square thumbnail visual changes as well.
Bunch of review
- Clean up more review.
Today:
- bug 983172 for the first
Julien
- On PTO till September 15th
Oleg
- bug 1048717 - [Messages][Refresh] Display call button at the header of single recipient thread
- Prepared patch and sent it for review (in review);
- Left some concern about non-valid numbers on Github.
- bug 1061682 - [Messages] Header shows wrong icons for back\close actions
- Verified that bug is fixed, asked Yan to ask for approval to uplift it in v2.1 :)
- bug 1061710 - [Flame][SMS] Unable to send sms
- I believe Steve is right that the following bug is the reason for this bug and for the one that Loli noticed, so tracking "bug 1061510 - [B2G] JavaScript Error: "ReadOnlyError: A mutation operation was attempted in a READ_ONLY transaction.".
- (Steve) Ya, there are bunch of bugs related to this issue(Anything related to settings DB is broken :p)
- (Oleg) Yep, this can break everything :)
- I believe Steve is right that the following bug is the reason for this bug and for the one that Loli noticed, so tracking "bug 1061510 - [B2G] JavaScript Error: "ReadOnlyError: A mutation operation was attempted in a READ_ONLY transaction.".
- bug 1059233 - [SMS] User cannot view the cropped image in SMS application and ActivityCanceled error was shown when try to view
- I see "storage is null" in the gallery when the issue happens, after that I can't even open gallery - "no memory card" error is displayed, attached logcat and redirected to Gaia::Gallery to reconfirm.
- (Steve) So it's only reproducible while no memory card inserted?
- (Oleg) Mm, I don't have memory card to check, but it always worked before and works after reboot...
- (Steve) Ya, I think so, but I can check for the memory card inserted scenario
- (Oleg) That would be nice, thanks!
- I see "storage is null" in the gallery when the issue happens, after that I can't even open gallery - "no memory card" error is displayed, attached logcat and redirected to Gaia::Gallery to reconfirm.
Other:
- Cleaned up review queue! :)
Today:
- Will handle review comments for the patches that currently in review;
- Will deal with review\feedback requests;
Day 4: 5th September
Steve
- bug 1048841 - [Messages] investigate performance hit from IndexedDB at startup (eg: Contact lookup)
- Waiting for the QC's reply. If they takes the gecko patch, we might need to provide the corresponding gaia patch to fix the profiling result
- bug 983172 - Parsing jpeg header information for downsampling the image for thumbnail
- Missing one image asset, and visual will update later. Trying to clean up the original hard-coded js styling as possible.
- (Oleg) yeah, that would be great!
- Missing one image asset, and visual will update later. Trying to clean up the original hard-coded js styling as possible.
Bunch of review
- Still on the reviews, has some concern in Eitan a11y's patch. Just thought that we create many element in JS runtime, and seems like we need to revisit this part in the future.
Today:
- Create patch for bug 983172
- I owe many reviews from Oleg, time to clean them up :p
- (Oleg) no worries! :)
Next week:
- Might take PTO for almost a week for some personal matter(you know that), I will still take review in the limited time, but might not able to send more patch :(
Julien
- On PTO till September 15th
Oleg
- bug 1061710 - [Flame][SMS] Unable to send sms
- Looks like the patch that intended to fix this is landed, will check once it appears on pvt.
- bug 1061215 - [Messages][Refactoring] Improve handling of remaining chars counter
- Prepared patch and asked for low-priority review (in review).
- bug 1053964 - [Messages][Tests] Add marionette JS test case to verify close\back button behaviour when app is in activity mode
- Added last fix with contenteditable=true as Steve noted and landed (landed).
- bug 1063043 - send button doesn't work when keyboard is open
- It's actually true when we tap not in the button center, will try to find out what is the reason, keyboard or some sort of click fluffing problem maybe.
- (Steve) I remember we seems doing some preventDefault action in send button to prevent keyboard dismiss on click long time ago... it seems like a workaround but maybe I'm wrong
- (Oleg) Ok, thanks for pointer! Will look into that too
- (Steve) Maybe I'm just wrong. Is it really hard to tap the send button? Since our send button is a little bit smaller then before
- (Oleg) I guess it's just habit, I always tap in center, so it didn't bother me before and now :)
- (Steve) If Dietrich is tapping on the "button container" but not button, maybe we could enlarge the button size for usability, but we need to confirm it first.
- (Oleg) The only thing that is strange and I'd like to confirm with reporter is, if he taps on the same area when keyboard is visible and when it's not. I assume that he's talking about the same area, if it's not because keyboard's iframe then it's something wrong in SMS. If I tap on bottom right corner for example when keyboard is shown - keyboard just becomes hidden, when I tap on the same area when keyboard is not visible, then it works as expected. I made preliminary debugging and don't see that click is fired on our button at all, maybe keyboard is capturing it somehow. don't know :) Just ideas. It's fine for Email, because button isn't too close to keyboard. The same is for Add attachment button
- (Steve) Will talk to kb owner then.
- (Oleg) Oh, cool, is he also in Taipei? :) Also just for the record (maybe not useful at all :)) if you debug keyboard app with Web IDE and select row-0 - web inspector shows it's boundary that a bit overlap our buttons, maybe it doesn't matter
Today:
- Will handle review comments for the patches that currently in review;
- Noticed that send button didn't work in browser - will take a look, I see exception from shared date helper about mozSettings, will check if it's related.
Day 5: 8th September
Steve
- Public Holiday in Taipei
Julien
- On PTO till September 15th
Oleg
- bug 1061710 - [Flame][SMS] Unable to send sms
- Asked reporter to check once again as settings dependency bug has been landed (awaiting info from reporter).
- bug 1063043 - send button doesn't work when keyboard is open
- Bisected gaia and noticed that first signs of that issue started to happen after "bug 1011087 - [B2G][SMS]Using edge gestures to change apps while keyboard is present, message bar is partially offscreen after returning to SMS app" landed. Everything works fine with "Edge swipe navigation" turned off. Asked Dietrich to confirm that turning off swipe navigation fixes issue for him. If so, will talk to Etienne (awaiting for info from reporter).
- bug 1050416 - [Messages][Build] Add build script to ignore the desktop-only folder for production build
- Prepared patch for it + some refactoring attempt in dekstop-only code and asked for low priority review (in review).
- bug 1063970 - [Messages][Refresh] Update styles of phone and url links located inside messages
- While dogfooding noticed that we didn't change styles for the links inside messages, asked Fang on what styles should be changed (awaiting info from VD).
Other:
- Played with Foxbox (https://github.com/gasolin/foxbox). Looks like vagrant+virtualbox+custom shell scripts is our way to go here. We probably just need to tune foxbox for our needs, but overall idea looks good to me. Will make a fork with some adaptations.
Today:
- Will handle review comments for the patches that currently in review;
- Clean up review queue.
Day 6: 9th September
Steve
- bug 1048841 - [Messages] investigate performance hit from IndexedDB at startup (eg: Contact lookup)
- bug 1038176 launch time latency issue seems able to be closed with gecko patch landed( \o/ ) So I might take some time to finish this nice-to-have item to make the
aggressiveness adjustable by device resolution.
- (Oleg) Yay! :)
- bug 983172 - Parsing jpeg header information for downsampling the image for thumbnail
- Patch given and some precious feedback returned. Will find some time to update the patch for review again.
- bug 1058459 - [SMS] Data shared from other applications are not shown as Drafts until the app is killed and open again
- I've been told that partner are eager to fix this problem in this sprint... Will try out some solution for this. Not sure if it's overkill to use localstorage simply for event bus, but worth to give it a try.
- (Oleg) Just attached, you can take a look if you have time - simple poc :) Didn't test much, just thrown what I had in mind :)
- (Steve) Maybe we could ask our partner for testing first.
- (Oleg) Yeah, and who is contact point from partner there? I'll try to test this patch a bit, ~1h just see major cases.
- (Steve) sasikala could help or even finish the patch if it works ;)
- (Oleg) Oh, cool :) It will save some time for us :) Will ask her then <-her Oh, got it :)
- I've been told that partner are eager to fix this problem in this sprint... Will try out some solution for this. Not sure if it's overkill to use localstorage simply for event bus, but worth to give it a try.
Bunch of review
- Eitan a11y's patch r+ and reply some review/ ni?.
Today:
- Update bug 983172
- Review deletion patch and call button first
Oleg
- bug 1063043 - send button doesn't work when keyboard is open
- Asked reporter to try to use send button without swipe navigation and prepared reduced test case for Etienne that demonstrates issue when we tap on button areas close to window edges (awaiting for info from reporter and Etienne).
- (Steve) So Etienne suspected that this might related to app edge swipe gesture?
- (Oleg) Nope, that I suspect, didn't get reply from Etienne yet, I've bisected that previously and also see the difference if I turn off swipe navigation...
- (Steve) ya, just try it out and the edge swipe option really affect the result... But it's nice catch ;)
- (Oleg) I hope this issue bothers Dietrich, not the other one that we haven't discovered yet :). Btw in test app click target is "Red container", but in our case it "messages-container", weeeeird ..
- Asked reporter to try to use send button without swipe navigation and prepared reduced test case for Etienne that demonstrates issue when we tap on button areas close to window edges (awaiting for info from reporter and Etienne).
- bug 1063970 - [Messages][Refresh] Update styles of phone and url links located inside messages
- While dogfooding noticed that we didn't change styles for the links inside messages, asked Fang on what styles should be changed (awaiting info from VD).
- (Steve) Thanks for discovering this... Missing this part in the VD spec
- (Oleg) yeah, my bad, noticed that when was looking for possible cases for "call" action in menu :)
- While dogfooding noticed that we didn't change styles for the links inside messages, asked Fang on what styles should be changed (awaiting info from VD).
- bug 1058459 - [SMS] Data shared from other applications are not shown as Drafts until the app is killed and open again
- Prepared small PoC, just was curious if that can work with localstorage as event bus - looks like it works - will attach to the bug and maybe it can give us some hints for better solution.
Other:
- Some reviews, new bug triaging and investigation.
Today:
- Will handle review comments for the patches that currently in review;
- Clean up review queue.
Julien
- On PTO till September 15th