Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Add merge link to duplicate contact popup #13081

Merged
merged 1 commit into from
Nov 14, 2018

Conversation

colemanw
Copy link
Member

Overview

Adds a merge button to the duplicate contact popup which was inadvertently removed in a7ba493

Before

screenshot from 2018-11-11 11-57-18

After

screenshot from 2018-11-11 11-54-17

@civibot
Copy link

civibot bot commented Nov 11, 2018

(Standard links)

@civibot civibot bot added the master label Nov 11, 2018
@eileenmcnaughton
Copy link
Contributor

from chat
screenshot 2018-11-13 12 01 59

@colemanw
Copy link
Member Author

@eileenmcnaughton I've updated the link to always keep the older contact regardless of which one we are viewing. That should be consistent with other merge links.

@pbarmak
Copy link

pbarmak commented Nov 14, 2018

@colemanw and @eileenmcnaughton I have tested this on our dev site, and it works well, it provides the link on a contact edit (and not on a new contact, which is good). Appreciate the work to get it back in.

@eileenmcnaughton eileenmcnaughton merged commit 4302d83 into civicrm:master Nov 14, 2018
@eileenmcnaughton eileenmcnaughton deleted the mergeBtn branch November 14, 2018 19:06
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants