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.

Sign up for GitHub

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

Jump to bottom

Hide 2FA status from other members in organization members list (#22999) #23023

Merged
merged 1 commit into from
Feb 21, 2023
Merged

Hide 2FA status from other members in organization members list (#22999) #23023

Changes from all commits
Commits
Show all changes
1 commit
Select commit
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Hide 2FA status from other members in organization members list (#22999) 
This is rather private information that should not be given to all
members in the same organization. Only show it to organization owners.
  • Loading branch information
@brechtvl @yardenshoham
brechtvl authored and yardenshoham committed Feb 20, 2023
commit c9295860ba9394c8470ad068600e3c87bd431d2d
2 changes: 2 additions & 0 deletions templates/org/member/members.tmpl
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -39,6 +39,7 @@
</div>
</div>
<div class="ui two wide column center">
{{if $.IsOrganizationOwner}}
<div class="meta">
{{$.locale.Tr "admin.users.2fa"}}
</div>
Expand All @@ -51,6 +52,7 @@
{{end}}
</strong>
</div>
{{end}}
</div>
{{end}}
<div class="ui three wide column">
Expand Down