Closed Bug 977528 Opened 6 years ago Closed 5 years ago

[Contacts][Feature] Importers should give information of automated merged contacts

Categories

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

defect

Tracking

(feature-b2g:2.0, tracking-b2g:backlog, b2g-v2.0 fixed)

VERIFIED FIXED
2.0 S3 (6june)
feature-b2g 2.0
tracking-b2g backlog
Tracking Status
b2g-v2.0 --- fixed

People

(Reporter: arcturus, Assigned: jmcf)

References

Details

(Keywords: feature, Whiteboard: [priority])

Attachments

(1 file)

+++ This bug was initially created as a clone of Bug #975691 +++

Right now the messages from an import operation (from sdcard, sim, or any other source), end with a message of the number of contacts read from that source.

Since version 1.2 we have automatic passive merging, which could lead to situations where  we don't have any contact in the list, read 12 contacts (from the sim card, a vcf, etc) and we end up in the list with (for example) 10.

The two 'missing' contacts are not really missing, since the algorithm for automatic merging was working behind scenes, merging for us contacts that we consider are the same (there are rules for that).

Basically the message says 12 contacts imported, and we should be more clear on the message. Julien has suggested something that works pretty well for me, it's extend that information with something like:

10 imported contacts
2 merged contacts

To keep the user informed of what really happened in the process
Summary: [Contacts][Feature Importers should give information of automated merged contacts → [Contacts][Feature] Importers should give information of automated merged contacts
ni to UX in order to know their PoV here. Thanks!
Flags: needinfo?(aymanmaat)
(In reply to Francisco Jordano [:arcturus] from comment #0)
> +++ This bug was initially created as a clone of Bug #975691 +++
> 
> Right now the messages from an import operation (from sdcard, sim, or any
> other source), end with a message of the number of contacts read from that
> source.
> 
> Since version 1.2 we have automatic passive merging, which could lead to
> situations where  we don't have any contact in the list, read 12 contacts
> (from the sim card, a vcf, etc) and we end up in the list with (for example)
> 10.
> 
> The two 'missing' contacts are not really missing, since the algorithm for
> automatic merging was working behind scenes, merging for us contacts that we
> consider are the same (there are rules for that).
> 
> Basically the message says 12 contacts imported, and we should be more clear
> on the message. Julien has suggested something that works pretty well for
> me, it's extend that information with something like:
> 
> 10 imported contacts
> 2 merged contacts
> 
> To keep the user informed of what really happened in the process


This is a very very good point and this lack of messaging is an oversight by me in the design of the merging functionality. We should prioritise this as it will lead to user orientation issues. 

From a UX PoV the messaging suggested here by Julien and Francisco is a suitable solution to keep the user informed and orientated. I would just propose a small alteration the wording as follows:

      10 imported contacts
      2 contacts merged
Flags: needinfo?(aymanmaat)
(In reply to ayman maat :maat from comment #2) 
> 
> From a UX PoV the messaging suggested here by Julien and Francisco is a
> suitable solution to keep the user informed and orientated. I would just
> propose a small alteration the wording as follows:
> 
>       10 imported contacts
>       2 contacts merged

In some cases (like FB, Gmail and Outlook), we could also remove friends, since we can 'unmark' contacts. So I guess we will have the possible:

10 imported contacts
2 contacts merged
10 contacts deleted

Perhaps a bit too much, but definitely will add all the info the user needs to know.
blocking-b2g: --- → 1.4?
blocking-b2g: 1.4? → backlog
Wilfred,

Please help get this on the roadmap for upcoming release.
blocking-b2g: backlog → -
Flags: needinfo?(wmathanaraj)
blocking-b2g: - → 1.5?
Flags: needinfo?(cawang)
included in 1.5 comms targetted
blocking-b2g: 1.5? → backlog
this is in the target list of 1.5
Flags: needinfo?(wmathanaraj)
Hi, 

I'd suggest showing 

12 imported contacts
Merge 2 duplicated contacts

If we show "10" as imported contacts, users might get confused because they think they already imported 12. Hence, I think we shall provide more details on this string.

However, I'm quite lost on the "delete" scenario. Can you please give me more information on this case? Thanks!
Flags: needinfo?(cawang)
Hi Carrie!

In the case of Fb, Gmail and outlook, we present a preview list of contacts to click on them to import, but also to click on them to remove from your list.

So, for example you can click in import contacts and unselect some of the contacts previously selected which will end up with us removing from the contacts list those contacts.

That's why I was suggesting to include the message for deletion as well.

Does it makes sense?
Flags: needinfo?(cawang)
Hi Francisco, 
I've been thinking about this question and found out the behaviour of deleting the contacts while deselect the imported ones is really weird. Even if we provide the string to inform users that some contacts are deleted, they will never figure out the rule. Hence, I'd suggest gray-out those imported contacts so that users won't deselect them and accidentally cause the deletion. If they want to delete any contacts, they shall edit it in Contact APP. What do you say? Is it doable?

In addition, I've check the grammar of the string. :p It should be:

12 imported contacts
2 duplicated contacts merged

Thanks!
Flags: needinfo?(cawang) → needinfo?(francisco.jordano)
Sorry! that should be

12 contacts imported
2 duplicated contacts merged

Thanks!
Hei Carrie,

now that we have the edit mode in contacts, your proposal makes sense to me.

So I'll suggest to keep this bug, do the new information display with imported and merged, and open a new bug to proceed with your suggestion about gray out the contacts already imported.
Flags: needinfo?(francisco.jordano)
See Also: → 999966
Just opened bug 999966 for removing that behavior.

Thanks.
Whiteboard: [priority]
Assignee: nobody → jmcf
If there is 12 contacts imported and none of them can be merged with existing contacts, then we display the string:

12 imported contacts

Thanks!
Attached file 19982.html
Attachment #8433924 - Flags: review?(francisco)
Comment on attachment 8433924 [details]
19982.html

Pretty awesome work Jose.

Left some minor comments on the PR.

Could we add unit tests to this please?
Attachment #8433924 - Flags: review?(francisco)
feature-b2g: --- → 2.0
Target Milestone: --- → 2.0 S3 (6june)
Attachment #8433924 - Flags: review?(francisco)
Comment on attachment 8433924 [details]
19982.html

Great work, and green travis :)
Attachment #8433924 - Flags: review?(francisco) → review+
https://github.com/mozilla-b2g/gaia/commit/d283b742a12ac43ec087f45b02d3817cf7ddab69
Status: NEW → RESOLVED
Closed: 5 years ago
Resolution: --- → FIXED
QA Contact: lolimartinezcr
Depends on: 1026370
Depends on: 1028106
Tested and working:
2.0
Hamachi
Gecko-e50f6b7
Gaia-3bfe47c
Status: RESOLVED → VERIFIED
blocking-b2g: backlog → ---
You need to log in before you can comment on or make changes to this bug.