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

chore: Update Administrator access wording #1142

Open
wants to merge 2 commits into
base: master
Choose a base branch
from
Open

Conversation

Pytal
Copy link
Member

@Pytal Pytal commented Oct 8, 2024

Fix #1141

image

Signed-off-by: Christopher Ng <chrng8@gmail.com>
@Pytal Pytal added this to the Nextcloud 31 milestone Oct 8, 2024
@Pytal Pytal requested a review from skjnldsv October 8, 2024 23:14
@Pytal Pytal self-assigned this Oct 8, 2024
@Pytal
Copy link
Member Author

Pytal commented Oct 8, 2024

/compile

@Pytal Pytal enabled auto-merge October 8, 2024 23:14
Signed-off-by: nextcloud-command <nextcloud-command@users.noreply.github.com>
Copy link
Member

@jancborchardt jancborchardt left a comment

Choose a reason for hiding this comment

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

See wording detail comment

@@ -6,6 +6,9 @@
<template>
<div class="admin">
<h3>{{ t('privacy', 'Administrators') }}</h3>
<p class="admin__desc">
{{ t('privacy', 'The following users are designated administrators and may potentially have access to your data through system storage or backups. However, they do not have easy access to your data via the web interface or client applications.') }}
Copy link
Member

Choose a reason for hiding this comment

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

Suggested change
{{ t('privacy', 'The following users are designated administrators and may potentially have access to your data through system storage or backups. However, they do not have easy access to your data via the web interface or client applications.') }}
{{ t('privacy', 'The following accounts are designated administrators and may potentially have access to your data through system storage or backups. However, they do not have easy access to your data via the web interface or client applications.') }}

Only super tiny wording detail. :) Otherwise a really good improvement!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Misleading text in access screen
3 participants