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

confidentiality clean up #1433

Draft
wants to merge 12 commits into
base: gnt
Choose a base branch
from
Draft

confidentiality clean up #1433

wants to merge 12 commits into from

Conversation

Swrup
Copy link
Contributor

@Swrup Swrup commented Mar 6, 2023

No description provided.

@hgouraud
Copy link
Collaborator

hgouraud commented Mar 6, 2023

is_empty_person should be a replacement for is_empty_name (aka ? ?).
Maybe is_unknown_person would be better??

I would rename is_hidden to is_empty_surname,( and possibly add is_empty_first_name!!).

@Swrup Swrup force-pushed the private branch 2 times, most recently from eb629df to 166c7f7 Compare May 11, 2023 08:29
Elie Canonici Merle and others added 9 commits June 14, 2023 10:05
  remove condition on wizard or friend from conf.hide_names
  report this change where conf.hide_private_names is used

  TODO: maybe use a "is_hidden" function instead where
  conf.hide_private_names is used; and make it check if
  more conditions;
  it looks buggy when we use conf.hide_private_names directly
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

Successfully merging this pull request may close these issues.

None yet

3 participants