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-21476 Rename clicks to unique clicks in mailing summary report #11319

Merged
merged 1 commit into from Nov 27, 2017

Conversation

JKingsnorth
Copy link
Contributor

@JKingsnorth JKingsnorth commented Nov 24, 2017

Overview

Rename 'Clicks' to 'Unique Clicks' in mailing summary report. Because it's showing 'the number of people who have clicked on links in the mailing' - not the total number of clicks.

The logic is correct. Just tidying up the labeling.

Before

Column in mailing summary is called 'Clicks'.

After

Column in mailing summary is called 'Unique clicks'.

Technical Details

No technical changes. String changed.


@mattwire
Copy link
Contributor

+1

@JKingsnorth
Copy link
Contributor Author

A test failure for changing a label! I think not. Jenkins, retest this please.

@eileenmcnaughton
Copy link
Contributor

I think the only review required for this is whether the change is an improvement - since it only adjusts the label text. I'm taking @mattwire thumb's up as confirmation of that.

@eileenmcnaughton eileenmcnaughton merged commit 9612152 into civicrm:master Nov 27, 2017
@JKingsnorth JKingsnorth deleted the CRM-21476 branch November 27, 2017 08:53
sluc23 pushed a commit to ixiam/civicrm-core that referenced this pull request Jan 10, 2018
CRM-21476 Rename clicks to unique clicks in mailing summary report
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
4 participants