Closed Bug 988243 Opened 10 years ago Closed 10 years ago

Multiple contact delete does not work with Facebook Contacts

Categories

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

ARM
Gonk (Firefox OS)
defect
Not set
major

Tracking

(blocking-b2g:2.0+, b2g-v2.0 verified)

VERIFIED FIXED
2.0 S1 (9may)
blocking-b2g 2.0+
Tracking Status
b2g-v2.0 --- verified

People

(Reporter: jmcf, Assigned: jmcf)

References

Details

Attachments

(1 file)

Multiple contact delete does not work with FB Contacts. We need to query the contact data before removing, in order to know it is a FB contact or not.
Pending functionality that needs to be addressed so nominating to 1.5?. Thanks!
Blocks: 956219
blocking-b2g: --- → 1.5?
Hi Jose,

I have already considered deleting facebook contacts in multiple contact delete.
Can you please share a scenario where its not working?

Regards
Ashay
(In reply to ashayb2g from comment #2)
> Hi Jose,
> 
> I have already considered deleting facebook contacts in multiple contact
> delete.
> Can you please share a scenario where its not working?
> 
> Regards
> Ashay

You are only deleting the local contact but not the FB Contact in the datastore. I will fix that
1.5+ to make sure it is fixed as part of the multiple contacts delete new feature
blocking-b2g: 1.5? → 1.5+
Attached file 17703.html
Attachment #8397753 - Flags: review?(francisco.jordano)
Comment on attachment 8397753 [details]
17703.html

Nice, this is the best solution and cleaner.

Tried and working well, thanks Jose, please merge once travis is green.

Thanks!
Attachment #8397753 - Flags: review?(francisco.jordano) → review+
landed in master: 

https://github.com/mozilla-b2g/gaia/commit/ebf839cbdd80de887b92a3d38b0dfd10d4b74608
Status: NEW → RESOLVED
Closed: 10 years ago
Keywords: verifyme
Resolution: --- → FIXED
Target Milestone: --- → 1.4 S5 (11apr)
Flags: in-testsuite+
Target Milestone: 1.4 S5 (11apr) → 2.0 S1 (9may)
Verifying Fixed.  Issue no longer occurs in Flame 2.0
Multiple Facebook contacts are able to be deleted properly.

Environmental Variables:
Device: Flame 2.0
BuildID: 20140912000202
Gaia: 91dd0e596aa7c124dd968e1474b23e7992dc35a1
Gecko: a66168598533
Version: 32.0 (2.0)
Firmware: V123
User Agent: Mozilla/5.0 (Mobile; rv:32.0) Gecko/32.0 Firefox/32.0
Status: RESOLVED → VERIFIED
QA Whiteboard: [QAnalyst-Triage?]
Flags: needinfo?(pbylenga)
Keywords: verifyme
QA Whiteboard: [QAnalyst-Triage?] → [QAnalyst-Triage+]
Flags: needinfo?(pbylenga)
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: