-
Notifications
You must be signed in to change notification settings - Fork 19
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
[WP#57970]Add warning when encryption for group folders are not enabled explicitly #713
base: release/2.7
Are you sure you want to change the base?
Conversation
c096a47
to
c1d0cc6
Compare
fccff3b
to
3dcc563
Compare
src/components/AdminSettings.vue
Outdated
@@ -182,6 +182,7 @@ | |||
:title="t('integration_openproject', 'Project folders (recommended)')" | |||
:is-setup-complete-without-project-folders="isSetupCompleteWithoutProjectFolders" | |||
:is-there-error-after-project-folder-and-app-password-setup="isThereErrorAfterProjectFolderAndAppPasswordSetup" | |||
:is-there-group-folders-encryption-warning="showEncryptionWarningForGroupFolders" |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
any reason the names don't match each other?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Yes it can be changed according to the function name. show-encryption-warning-for-groupfolders
src/components/AdminSettings.vue
Outdated
@@ -265,6 +266,12 @@ | |||
</p> | |||
<p class="note-card--error-description" v-html="projectFolderSetUpErrorMessageDescription(state.project_folder_info.errorMessage)" /> <!-- eslint-disable-line vue/no-v-html --> | |||
</NcNoteCard> | |||
<NcNoteCard v-else-if="showEncryptionWarningForGroupFolders" class="note-card" type="warning"> | |||
<p class="note-card--title"> | |||
<b>{{ t('integration_openproject', 'Encryption for Group Folders are not enabled.') }}</b> |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
<b>{{ t('integration_openproject', 'Encryption for Group Folders are not enabled.') }}</b> | |
<b>{{ t('integration_openproject', 'Encryption for the Group Folders App is not enabled.') }}</b> |
src/components/AdminSettings.vue
Outdated
<p class="note-card--title"> | ||
<b>{{ t('integration_openproject', 'Encryption for Group Folders are not enabled.') }}</b> | ||
</p> | ||
<p class="note-card--warning-description" v-html="getGroupFolderEncryptionWarningHint" /> <!-- eslint-disable-line vue/no-v-html --> |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
<p class="note-card--warning-description" v-html="getGroupFolderEncryptionWarningHint" /> <!-- eslint-disable-line vue/no-v-html --> | |
<p class="note-card--warning-description" v-html="getGroupFoldersEncryptionWarningHint" /> <!-- eslint-disable-line vue/no-v-html --> |
be consequent with singular/plural
src/components/AdminSettings.vue
Outdated
getGroupFolderEncryptionWarningHint() { | ||
const linkText = t('integration_openproject', 'documentation') | ||
const htmlLink = `<a class="link" href="https://www.openproject.org/docs/system-admin-guide/integrations/nextcloud/#files-are-not-encrypted-when-using-nextcloud-server-side-encryption" target="_blank" title="${linkText}">${linkText}</a>` | ||
return t('integration_openproject', 'Server-side encryption is active, but encryption for group folders is not yet enabled. To ensure secure storage of files in project folders, please follow the configuration steps in the {htmlLink}.', { htmlLink }, null, { escape: false, sanitize: false }) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
here group folders
is written in lower case, please be consequent when referring to the Group Folders App
Hello there, We hope that the review process is going smooth and is helpful for you. We want to ensure your pull request is reviewed to your satisfaction. If you have a moment, our community management team would very much appreciate your feedback on your experience with this PR review process. Your feedback is valuable to us as we continuously strive to improve our community developer experience. Please take a moment to complete our short survey by clicking on the following link: https://cloud.nextcloud.com/apps/forms/s/i9Ago4EQRZ7TWxjfmeEpPkf6 Thank you for contributing to Nextcloud and we hope to hear from you soon! (If you believe you should not receive this message, you can add yourself to the blocklist.) |
Signed-off-by: Sagar <[email protected]>
Signed-off-by: Sagar <[email protected]>
Signed-off-by: Sagar <[email protected]>
Signed-off-by: Sagar <[email protected]>
Signed-off-by: nabim777 <[email protected]>
49d3d38
to
d5d24db
Compare
PHP Code CoverageCoverage after merging add/warnForGroupFolderEncryption into release/2.7 will be
Coverage Report
|
Description
This PR:
Related Issue or Workpackage
Screenshots (if appropriate):
Types of changes
Checklist:
CHANGELOG.md
file