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

Rename Non-protected variables #17

Open
brodrigu opened this issue Jul 24, 2020 · 0 comments
Open

Rename Non-protected variables #17

brodrigu opened this issue Jul 24, 2020 · 0 comments

Comments

@brodrigu
Copy link

In Reporting in Sparrow, you define non-protected variables:

Non-protected variables
Many variables are relevant for one player only, like ad labels (click, view, click id), ABTest ID (see ABTest section below), and the interest group for the advertiser. These variables should be hidden to the other players and do not need other specific attention in this report, they do not represent any risk with regard to the user privacy and can be reported as is per display.

Non-protected suggests these variables are available to anyone.

I would suggest using private to align with class variable types or perhaps hidden or single-party to more accurately describe how they are treated.

@brodrigu brodrigu changed the title Rename Rename Non-protected variables Jul 24, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

1 participant