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

CRM-20841 - Dedupe - Show on_hold, is_bulkmail or signature merge… #10630

Merged
merged 1 commit into from
Jun 6, 2018

Conversation

tschuettler
Copy link
Contributor

@tschuettler tschuettler commented Jul 10, 2017

… conflicts

https://issues.civicrm.org/jira/projects/CRM/issues/CRM-20841

Overview

Displays the on_hold, bulkmail and signature status along the email adress in a merge form. If the email together its status differs between original and duplicate contact it is visually highlighted as a conflict.

Before

It is not shown that one of the emails is set to on hold and thus you may unitentionally overwrite that status.
grafik

After

grafik

Technical Details

It is using the same technique as for the address handling.

Comments

This PR only handles checking whether a signature exists or not, but does not show a conflict if signatures only differ in their contents.

@tschuettler
Copy link
Contributor Author

This PR only handles checking whether a signature exists or not, but does not show a conflict if signatures only differ in their contents.

@tschuettler tschuettler changed the title CRM-CRM-20841 - Dedupe - Show on_hold, is_bulkmail or signature merge… CRM-20841 - Dedupe - Show on_hold, is_bulkmail or signature merge… Aug 11, 2017
@eileenmcnaughton
Copy link
Contributor

tested this & it seems like a display improvement without any noticeable downside

@eileenmcnaughton eileenmcnaughton merged commit 1c5bc02 into civicrm:master Jun 6, 2018
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.

2 participants