Closed Bug 811658 Opened 12 years ago Closed 11 years ago

Adress info from FB is showing hometown but not current living place

Categories

(Firefox OS Graveyard :: Gaia, defect, P3)

x86
Gonk (Firefox OS)
defect

Tracking

(blocking-basecamp:+)

RESOLVED FIXED
B2G C1 (to 19nov)
blocking-basecamp +

People

(Reporter: jmcf, Assigned: jmcf)

References

Details

(Whiteboard: QA Regress exclude)

Attachments

(2 files)

We agreed with UX to show the current location of the FB friend and in case that it is not defined we will be showing the hometown as it was previously. Finally if none of them are defined nothing will be shown
Status: NEW → ASSIGNED
Attached file GH PR 6391
Attachment #681420 - Flags: review?(crdlc)
Attachment #681420 - Attachment is patch: true
Attachment #681420 - Attachment mime type: text/html → text/plain
Attachment #681420 - Flags: review?(crdlc) → review+
Attachment #681420 - Attachment is patch: false
Attachment #681420 - Attachment mime type: text/plain → text/html
Comment on attachment 681420 [details]
GH PR 6391

NOTE: If blocking-basecamp+ is set, just land it for now.

[Approval Request Comment]
Bug caused by (feature/regressing bug #): 
User impact if declined: 
Testing completed: 
Risk to taking this patch (and alternatives if risky):
Attachment #681420 - Flags: approval-gaia-master?(francisco.jordano)
Reviewer's changes have been addressed in the PR. Francisco, please check
Attachment #681420 - Flags: approval-gaia-master?(francisco.jordano) → approval-gaia-master+
Status: ASSIGNED → RESOLVED
Closed: 12 years ago
Resolution: --- → FIXED
blocking-basecamp: ? → +
Issue verified on Build 20130103070201
Issue repro's on Unagi build 201301120702022.
See attached screenshot.
In this example Matt Qanalyst is set up with 
Lives in Kirkland, Washington
From Seattle, Washington.

Unagi is only showing the following:
HOME
Kirkland, Wa.
Status: RESOLVED → REOPENED
Resolution: FIXED → ---
Attached image Only showing Home
Please file a followup bug and link it as a dependency. Don't reopen - this makes tracking blockers a pain if we reopen.
Status: REOPENED → RESOLVED
Closed: 12 years ago11 years ago
Resolution: --- → FIXED
Blocks: 830020
No longer blocks: 830020
Depends on: 830020
This bug depends on 830020.
Whiteboard: QA Regress exclude
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: