Closed
Bug 1037338
Opened 10 years ago
Closed 10 years ago
Installed apps arent added to smart collection
Categories
(Firefox OS Graveyard :: Gaia::Everything.me, defect)
Tracking
(blocking-b2g:2.0+, b2g-v2.0 verified, b2g-v2.1 verified)
People
(Reporter: daleharvey, Assigned: crdlc)
References
Details
(Whiteboard: [systemsfe])
Attachments
(3 files)
STR: 1. Install facebook 2. Open 'Social' smart collection Facebook should be part of the pinned apps, it isnt
Reporter | ||
Comment 1•10 years ago
|
||
If I wrap a .then around https://github.com/mozilla-b2g/gaia/blob/master/apps/collection/js/native_info.js#L76 then facebook is displayed the second time I open the smart collection
Reporter | ||
Updated•10 years ago
|
Assignee: nobody → dale
Assignee | ||
Comment 2•10 years ago
|
||
I did it to avoid the race condition when the user adds an icon to a collection and he tries to open the collection immediately https://github.com/mozilla-b2g/gaia/blob/master/apps/collection/js/view_apps.js#L62 Maybe you can follow the same approach
Assignee | ||
Comment 3•10 years ago
|
||
See bug 1029766 because Kevin changed a bit the logic
See Also: → 1029766
Reporter | ||
Comment 4•10 years ago
|
||
Talked to Cristian on IRC and he said he will steal this one
Assignee: dale → crdlc
Assignee | ||
Updated•10 years ago
|
Status: NEW → ASSIGNED
Assignee | ||
Updated•10 years ago
|
Whiteboard: [systemsfe]
Assignee | ||
Comment 5•10 years ago
|
||
Hi mates, it seems that works fine but I have to leave the office right now and I prefer a feedback on time before the Monday's review
Attachment #8454394 -
Flags: feedback?(kgrandon)
Attachment #8454394 -
Flags: feedback?(dale)
Comment 6•10 years ago
|
||
Comment on attachment 8454394 [details]
Github pull request
Seems like it should work fine to me, thanks! (Needs a rebase)
Attachment #8454394 -
Flags: feedback?(kgrandon) → feedback+
Updated•10 years ago
|
blocking-b2g: --- → 2.0?
QA Whiteboard: [VH-FL-blocking-][VH-FC-blocking+]
Comment 7•10 years ago
|
||
This was working at one point. I'd like to add an integration test for this, which will require mocking the "native info" call from E.me within our test fixtures. Cristian - If you don't get around to it, please leave this bug open to track that work, or spin up another bug to add a test.
Reporter | ||
Comment 8•10 years ago
|
||
It still works but is fairly intermittent, I was reproducing it every time at some point, now I cant reproduce it at all
Comment 9•10 years ago
|
||
Can we get a clarification on how intermittent?
blocking-b2g: 2.0? → ---
QA Whiteboard: [VH-FL-blocking-][VH-FC-blocking+] → [VH-FL-blocking-][VH-FC-blocking?]
Keywords: qawanted
Comment 10•10 years ago
|
||
(In reply to Dale Harvey (:daleharvey) from comment #8) > It still works but is fairly intermittent, I was reproducing it every time > at some point, now I cant reproduce it at all I wonder if it could be possibly related to server response times? It seems like we're missing the init() call here, so it could be possible that sometimes the server responds fast enough, and sometimes it doesn't?
Comment 11•10 years ago
|
||
I can reproduce this issue 100% if I open the Social Smart Collection before installing Facebook (on a freshly flashed Flame). Facebook now is appearing below the pinned app section, even after restarting the phone. However, Facebook will appear with the pinned apps if it is installed before the user opens Social Smart Collection for the first time. Facebook will appear with the pinned apps even after it is deleted if user installed it the first time before opening Social Smart Collections. Resetting the phone though Settings > Device Information allows for the different outcomes. Flame Master BuildID: 20140711040202 Gaia: c47094a26c87ba71a3da4bae54febd0da21f3393 Gecko: 1b1296d00330 Version: 33.0a1 (Master) Firmware Version: v122 User Agent: Mozilla/5.0 (Mobile; rv:33.0) Gecko/33.0 Firefox/33.0
QA Whiteboard: [VH-FL-blocking-][VH-FC-blocking?] → [VH-FL-blocking-][VH-FC-blocking?] [QAnalyst-Triage?]
status-b2g-v2.1:
--- → affected
Flags: needinfo?(jmitchell)
Keywords: qawanted
Updated•10 years ago
|
QA Whiteboard: [VH-FL-blocking-][VH-FC-blocking?] [QAnalyst-Triage?] → [VH-FL-blocking-][VH-FC-blocking?] [QAnalyst-Triage+]
Flags: needinfo?(jmitchell)
Comment 12•10 years ago
|
||
I can reproduce only with OWD user builds, which ship the Social collection pre-installed at about 30%. When the pinning fails, I see this in the log: E/GeckoConsole( 1074): Content JS ERROR at app://collection.gaiamobile.org/gaia_build_defer_synchronize.js:320 in error: evme ERROR NativeInfo error TypeError hope this helps :)
Assignee | ||
Comment 13•10 years ago
|
||
No server issues. Two scenarios: * The synchronization process is performed alone (without opening smart collection), it works fine because the icon is not dirty at this point [1] * Users open the smart collection immediately while or before synchronization process, so the icon is dirty here [1] and when the code tries to render the icon, the homeicon was not initialized here [2] and it fails in line 353 as Amir said in comment 12 [1] https://github.com/mozilla-b2g/gaia/blob/master/apps/collection/js/objects.js#L102 [2] https://github.com/mozilla-b2g/gaia/blob/master/apps/collection/js/objects.js#L265
Assignee | ||
Updated•10 years ago
|
OS: Mac OS X → Gonk (Firefox OS)
Hardware: x86 → All
Comment 14•10 years ago
|
||
(In reply to Eric Kramer [:Kramer] from comment #11) > I can reproduce this issue 100% if I open the Social Smart Collection before > installing Facebook (on a freshly flashed Flame). Facebook now is appearing > below the pinned app section, even after restarting the phone. Also reproduced with these STR on 2.0 today's build. Gaia ca022f811bcbbda0f89086094a9e92bb220fea18 Gecko https://hg.mozilla.org/releases/mozilla-aurora/rev/376889ab0e02 BuildID 20140713160202 Version 32.0a2 ro.build.version.incremental=109 ro.build.date=Mon Jun 16 16:51:29 CST 2014 B1TC00011220
blocking-b2g: --- → 2.0?
QA Whiteboard: [VH-FL-blocking-][VH-FC-blocking?] [QAnalyst-Triage+] → [VH-FL-blocking-][VH-FC-blocking+] [QAnalyst-Triage+]
status-b2g-v2.0:
--- → affected
Updated•10 years ago
|
blocking-b2g: 2.0? → 2.0+
Assignee | ||
Comment 15•10 years ago
|
||
Comment on attachment 8454394 [details]
Github pull request
addressed comments and tested
Attachment #8454394 -
Flags: review?(kgrandon)
Updated•10 years ago
|
Target Milestone: --- → 2.0 S6 (18july)
Comment 16•10 years ago
|
||
Comment on attachment 8454394 [details]
Github pull request
I think this is a good first step. I'm not convinced that we're fully out of the woods on the native info stuff, but I think we can land this initially.
Attachment #8454394 -
Flags: review?(kgrandon)
Attachment #8454394 -
Flags: review+
Attachment #8454394 -
Flags: feedback?(dale)
Comment 17•10 years ago
|
||
Some more fixes as I noticed some errors when we had to generate the icon. I'm not convinced this is fully working yet though. Cristian - if you want to include this in your PR you can, or I can finish it up separately.
Comment 18•10 years ago
|
||
Comment on attachment 8455211 [details] [review] Part 2 - More native info fixes So it appears that my patch fixes saving when it needs to update the icon, but still does not update the icon. I think it's fine to land as-is, as the icon problems are known and being tracked in another non-blocking bug somewhere. Cristian - what do you think?
Attachment #8455211 -
Flags: review?(crdlc)
Assignee | ||
Comment 19•10 years ago
|
||
Comment on attachment 8455211 [details] [review] Part 2 - More native info fixes I gonna add it to the main pr
Attachment #8455211 -
Flags: review?(crdlc) → review+
Assignee | ||
Comment 20•10 years ago
|
||
added your patch
Assignee | ||
Comment 21•10 years ago
|
||
The icon is not updated although the user does not try to open the smart collection during the synchronization. Obviously it is a different bug thought (In reply to Kevin Grandon :kgrandon from comment #18) > Comment on attachment 8455211 [details] [review] > Part 2 - More native info fixes > > So it appears that my patch fixes saving when it needs to update the icon, > but still does not update the icon. I think it's fine to land as-is, as the > icon problems are known and being tracked in another non-blocking bug > somewhere. > > Cristian - what do you think?
Assignee | ||
Comment 22•10 years ago
|
||
Merged in master: https://github.com/crdlc/gaia/commit/32982f4f83474d08135e97e0e8870f14a65f967b
Status: ASSIGNED → RESOLVED
Closed: 10 years ago
Resolution: --- → FIXED
Assignee | ||
Updated•10 years ago
|
Comment 23•10 years ago
|
||
v2.0: https://github.com/mozilla-b2g/gaia/commit/2604316275581bfe261de1e8b1cc5069f7f89606
Comment 24•10 years ago
|
||
(In reply to Kevin Grandon :kgrandon from comment #16) > I'm not convinced that we're fully out of > the woods on the native info stuff, but I think we can land this initially. I agree. I feel the NativeInfo stuff is prone to errors. It was done very quickly to meet v2.0 FL deadline. Any suggestions on how to make it better are welcome :)
Comment 25•10 years ago
|
||
This issue has been verified successfully on Flame 2.0 & 2.1 See attachment: Verify_video.MP4 Reproducing rate: 0/3 Reproducing steps: 1.Launch Marketplace. 2.Download and install Facebook. 3.Press Home key and launch Social. ** The Facebook is display in Social smart collection screen Flame2.0 build: Gaia-Rev 8d1e868864c8a8f1e037685f0656d1da70d08c06 Gecko-Rev https://hg.mozilla.org/releases/mozilla-b2g32_v2_0/rev/c756bd8bf3c3 Build-ID 20141127000203 Version 32.0 Flame2.1 build: Gaia-Rev 5372b675e018b6aac97d95ff5db8d4bd16addb9b Gecko-Rev https://hg.mozilla.org/releases/mozilla-b2g34_v2_1/rev/f34377ae402b Build-ID 20141127001201 Version 34.0
Updated•10 years ago
|
You need to log in
before you can comment on or make changes to this bug.
Description
•