PPL

merge contacts should merge stories, photos and multi emails

Bug #716795 reported by Bassel Safadi
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
PPL
Fix Released
Critical
Bassel Safadi

Bug Description

currently stories and photos are not merged..
also if two accounts with same name and two different emails are merged, the second email is lost..

Bassel Safadi (bassel)
Changed in ppl:
assignee: nobody → Bassel Safadi (bassel)
importance: Undecided → High
Bassel Safadi (bassel)
Changed in ppl:
status: New → Confirmed
Bassel Safadi (bassel)
Changed in ppl:
milestone: none → 0.2
Changed in ppl:
importance: High → Critical
Revision history for this message
Bassel Safadi (bassel) wrote :

Merge now updates stories and portraits

Changed in ppl:
status: Confirmed → In Progress
Revision history for this message
Bassel Safadi (bassel) wrote :

it add the other emails as well now. the other_emails field is not displayed anywhere though, it's the matter of finding a place for it and then using a split function to show them.

Christopher any suggestions on this?

Changed in ppl:
status: In Progress → Fix Released
Revision history for this message
Christopher Adams (christopheradams) wrote : Re: [Bug 716795] Re: merge contacts should merge stories, photos and multi emails

Does that mean just multi emails left?

On Feb 13, 2011 1:45 PM, "Bassel Safadi" <email address hidden> wrote:

Merge now updates stories and portraits

** Changed in: ppl
      Status: Confirmed => In Progress

--
You received this bug notification because you are a member of PPL
Developers, which is subscri...
 In Progress

Bug description:
currently stories and photos are not merged..
also if two accounts with same n...

Revision history for this message
Bassel Safadi (bassel) wrote :

tested this and it works. others please test and confirm

Revision history for this message
Bassel Safadi (bassel) wrote :

no I mean multi emails are stored now. but there is currently no place inside the user profile page to display the multi emails

Revision history for this message
Christopher Adams (christopheradams) wrote :

Ok, displaying multi emails is a separate issue.

I will test soon and commit your code.

On Feb 13, 2011 2:20 PM, "Bassel Safadi" <email address hidden> wrote:

no I mean multi emails are stored now. but there is currently no place
inside the user profile page to display the multi emails

--
You received this bug notification because you are a member of PPL
Developers, which is subscri...
 Fix Released

Bug description:
currently stories and photos are not merged..
also if two accounts with same n...

To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.