Skip to content

Updated Identifier breakdown information #4263

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

Merged
merged 7 commits into from
Mar 30, 2023
Merged

Conversation

Panandhan22
Copy link
Contributor

@Panandhan22 Panandhan22 commented Feb 21, 2023

Proposed changes

Identifier Breakdown

The audience summary gives a breakdown of the percentages of external_ids of users in the audience. These are the default ID's that has been included in Identity resolution configuration. It helps us by showing the portion of the audience has which identities on it and is used to verify the audience size/profiles are correct and as expected.

info ""
The Identifier breakdown won't show custom ID's which has been included in identity resolution configuration. Only default external IDs will be shown in Identifier breakdown.

Merge timing

Related issues (optional)

Panandhan22 and others added 2 commits March 17, 2023 15:25
Changed redaction of a few sentences to make it clearer for clients
@Panandhan22 Panandhan22 marked this pull request as ready for review March 29, 2023 07:50
Copy link
Contributor

@rchinn1 rchinn1 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks for submitting @Panandhan22! Left a few suggestions, mainly to update passive voice to active and change casing to match the UI.

@rchinn1 rchinn1 merged commit bc765b1 into develop Mar 30, 2023
@rchinn1 rchinn1 deleted the Panandhan22-patch-1 branch March 30, 2023 16:20
@github-actions
Copy link
Contributor

Thank you for your contribution! Your pull request is merged, but may take a day or two to appear on the site.

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