1036355 - [Collection App] Collections are not updated after installing bookmarks from browser

Status: RESOLVED WONTFIX

(Firefox OS Graveyard :: Gaia::Everything.me, defect, P3)

Description

[Cristian Rodriguez (:crdlc)]

STR:

1) Go to homescreen
2) Open http://www.facebook.com
3) Click on the star icon
3) Add this bookmark to homescreen

Expected:

Social collection has this new bookmark in its list of pinned icons

Result:

No bookmark in social smart collection

-----

E/GeckoConsole( 7352): [JavaScript Error: "TypeError: GaiaGrid.Bookmark is not a constructor" {file: "app://collection.gaiamobile.org/js/home_icons.js" line: 63}]

After fixing this dependency I can see how ev.me makes the query but the response is an empty object

Comment 1

[Cristian Rodriguez (:crdlc)]

Hi Amir, if there is a problem in the server, after fixing it there, I could fix the dependency in Gaia. Thanks a lot

Comment 2

[Jason Smith [:jsmith]]

QA Wanted - Was this working in the old e.me search in 1.4?

Comment 3

[Cody Roesch [:croesch]]

This bug repro's on: Flame 2.1 Master, Flame 2.0, Flame 1.4 and Buri 2.1

Actual Results: Adding Facebook.com bookmark to the homescreen does NOT add it to the pinned apps in the Social Collection.

Environmental Variables:
Device: Flame Master
Build ID: 20140708201331
Gaia: 0f9f11d0a6dadb3ea27160204bbe911c1ad69a6f
Gecko: 196d05832e12
Version: 33.0a1 (Master)
Firmware Version: v122
-----------------------------------------------
Environmental Variables:
Device: Flame 2.0
Build ID: 20140708163231
Gaia: 1dd043857399c713e3b509c0ed31bdf20326f08b
Gecko: 0b5f757da75a
Version: 32.0a2 (2.0)
Firmware Version: v122
-----------------------------------------------
Environmental Variables:
Device: Flame 1.4
Build ID: 20140708070432
Gaia: 229864ff4ad90899f017341b9e81ed0b53498c66
Gecko: 7e146a1d7542
Version: 30.0 (1.4)
Firmware Version: v122
-----------------------------------------------
Environmental Variables:
Device: Buri Master
Build ID: 20140708201331
Gaia: 0f9f11d0a6dadb3ea27160204bbe911c1ad69a6f
Gecko: 196d05832e12
Version: 33.0a1 (Master)
Firmware Version: v1.2device.cfg

Comment 4

[Amir Nissim (:amirn)]

I double checked with our server guys. The NativeInfo api does not support BookmarkURLs as guids.
Sorry for confusing you Crisitan :)

Comment 5

[Cristian Rodriguez (:crdlc)]

Closing because it is not supported in 1.4. My mistake was when I read this https://github.com/mozilla-b2g/gaia/blob/master/apps/collection/js/native_info.js#L48

I will fill a new bug to remove the call to 'install' for bookmarks from vertical home because it is not supported

Comment 7

[Peter Dolanjski [:pdol]]

Reopening to add fixing this this to the backlog.

Comment 8

[Kevin Grandon :kgrandon]

Mass update: Resolve/Wontfix all existing collections bugs as this feature is now removed.

Please re-open or file a new bug if you feel that this bug still exists in master.