Adjust the documentation link according to review

Signed-off-by: sagargurung1001@gmail.com <sagargurung1001@gmail.com>
nextcloud · Mar 25, 2024 · 57c41ee · 57c41ee
1 parent d9a7f71
commit 57c41ee
Show file tree

Hide file tree

Showing 4 changed files with 8 additions and 13 deletions.
diff --git a/src/components/AdminSettings.vue b/src/components/AdminSettings.vue
@@ -1,7 +1,7 @@
 <template>
 	<div id="openproject_prefs" class="section">
 		<TermsOfServiceUnsigned :is-all-terms-of-service-signed-for-user-open-project="isAllTermsOfServiceSignedForUserOpenProject" />
-		<SettingsTitle :show-integration-setup-link-information="showIntegrationSetupLinkInformation" is-setting="admin" />
+		<SettingsTitle is-setting="admin" />
 		<div class="openproject-server-host">
 			<FormHeading index="1"
 				:title="t('integration_openproject', 'OpenProject server')"
@@ -434,7 +434,6 @@ export default {
 			isFormStep: null,
 			isDarkTheme: null,
 			isAllTermsOfServiceSignedForUserOpenProject: true,
-			showIntegrationSetupLinkInformation: true,
 			userSettingDescription: USER_SETTINGS,
 		}
 	},
@@ -583,7 +582,6 @@ export default {
 				}
 				if (this.state.openproject_instance_url && this.state.openproject_client_id && this.state.openproject_client_secret && this.state.nc_oauth_client) {
 					this.showDefaultManagedProjectFolders = true
-					this.showIntegrationSetupLinkInformation = false
 				}
 				if (this.state.openproject_instance_url) {
 					this.formMode.server = F_MODES.VIEW

diff --git a/src/components/settings/SettingsTitle.vue b/src/components/settings/SettingsTitle.vue
@@ -27,10 +27,6 @@ export default {
 			type: Boolean,
 			required: false,
 		},
-		showIntegrationSetupLinkInformation: {
-			type: Boolean,
-			required: false,
-		},
 	},
 	computed: {
 		title() {
@@ -42,12 +38,12 @@ export default {
 			return t('integration_openproject', 'Visit our documentation for in-depth information on {htmlLink} integration.', { htmlLink }, null, { escape: false, sanitize: false })
 		},
 		getUserGuideDocumentationLinkText() {
-			const linkText = t('integration_openproject', 'Nextcloud integration user guide')
+			const linkText = t('integration_openproject', 'user guide')
 			const htmlLink = `<a class="link" href="https://www.openproject.org/docs/user-guide/file-management/nextcloud-integration/" target="_blank" title="${linkText}">${linkText}</a>`
-			return t('integration_openproject', 'Please go to our {htmlLink} to learn more about how to work with the OpenProject integration.', { htmlLink }, null, { escape: false, sanitize: false })
+			return t('integration_openproject', 'Learn how to get the most out of the OpenProject integration by visiting our {htmlLink}.', { htmlLink }, null, { escape: false, sanitize: false })
 		},
 		sanitizedHintText() {
-			if (this.isSetting === 'admin' && this.showIntegrationSetupLinkInformation) {
+			if (this.isSetting === 'admin') {
 				return dompurify.sanitize(this.getSetUpIntegrationDocumentationLinkText, { ADD_ATTR: ['target'] })
 			} else if (this.isSetting === 'personal' && this.isConnectedWithOpenproject) {
 				return dompurify.sanitize(this.getUserGuideDocumentationLinkText, { ADD_ATTR: ['target'] })
@@ -80,6 +76,7 @@ export default {
 
 .settings .link {
 	color: #1a67a3 !important;
+	font-style: normal;
 }
 
 </style>
diff --git a/tests/jest/components/AdminSettings.spec.js b/tests/jest/components/AdminSettings.spec.js
@@ -267,10 +267,10 @@ describe('AdminSettings.vue', () => {
 			expect(setupIntegrationDocumentationLink.text()).toBe('Visit our documentation for in-depth information on {htmlLink} integration.')
 		})
 
-		it('should not be visible when integration is completed', () => {
+		it('should be visible when integration is completed', () => {
 			const wrapper = getMountedWrapper({ state: completeIntegrationState })
 			const setupIntegrationDocumentationLink = wrapper.find(selectors.setupIntegrationDocumentationLinkSelector)
-			expect(setupIntegrationDocumentationLink.text()).toBe('')
+			expect(setupIntegrationDocumentationLink.text()).toBe('Visit our documentation for in-depth information on {htmlLink} integration.')
 		})
 	})
 

diff --git a/tests/jest/components/PersonalSettings.spec.js b/tests/jest/components/PersonalSettings.spec.js
@@ -100,7 +100,7 @@ describe('PersonalSettings.vue', () => {
 				it('should show not show user guide documentation link', () => {
 					const wrapper = getMountedWrapper({ state: { user_name: 'test', token: '123', admin_config_ok: true } })
 					const userGuideIntegrationDocumentationLink = wrapper.find(userGuideIntegrationDocumentationLinkSelector)
-					expect(userGuideIntegrationDocumentationLink.text()).toBe('Please go to our {htmlLink} to learn more about how to work with the OpenProject integration.')
+					expect(userGuideIntegrationDocumentationLink.text()).toBe('Learn how to get the most out of the OpenProject integration by visiting our {htmlLink}.')
 				})
 			})