fix: Make the High-performance backend warnings more actionable

Signed-off-by: Joas Schilling <[email protected]>

lib/Settings/Admin/AdminSettings.php

@@ -25,6 +25,7 @@
 use OCP\IUserSession;
 use OCP\L10N\IFactory;
 use OCP\Settings\ISettings;
+use OCP\Support\Subscription\IRegistry;
 use OCP\Util;
 
 class AdminSettings implements ISettings {
@@ -38,6 +39,7 @@ public function __construct(
 		private ICacheFactory $memcacheFactory,
 		private IGroupManager $groupManager,
 		private MatterbridgeManager $bridgeManager,
+		private IRegistry $subscription,
 		IUserSession $userSession,
 		private IL10N $l10n,
 		private IFactory $l10nFactory,
@@ -126,6 +128,7 @@ protected function initTurnServers(): void {
 	}
 
 	protected function initSignalingServers(): void {
+		$this->initialState->provideInitialState('has_valid_subscription', $this->subscription->delegateHasValidSubscription());
 		$this->initialState->provideInitialState('has_cache_configured', $this->memcacheFactory->isAvailable());
 		$this->initialState->provideInitialState('signaling_mode', $this->talkConfig->getSignalingMode(false));
 		$this->initialState->provideInitialState('signaling_servers', [

lib/SetupCheck/HighPerformanceBackend.php

@@ -16,6 +16,7 @@
 use OCP\IURLGenerator;
 use OCP\SetupCheck\ISetupCheck;
 use OCP\SetupCheck\SetupResult;
+use OCP\Support\Subscription\IRegistry;
 
 class HighPerformanceBackend implements ISetupCheck {
 	public function __construct(
@@ -24,6 +25,7 @@ public function __construct(
 		readonly protected IURLGenerator $urlGenerator,
 		readonly protected IL10N $l,
 		readonly protected Manager $signalManager,
+		readonly protected IRegistry $subscription,
 	) {
 	}
 
@@ -39,11 +41,16 @@ public function run(): SetupResult {
 		if ($this->talkConfig->getSignalingMode() === Config::SIGNALING_INTERNAL) {
 			$setupResult = SetupResult::error(...);
 			if ($this->talkConfig->getHideSignalingWarning()) {
-				$setupResult = SetupResult::warning(...);
+				$setupResult = SetupResult::info(...);
 			}
+			$documentation = 'https://nextcloud-talk.readthedocs.io/en/latest/quick-install/';
+			if ($this->subscription->delegateHasValidSubscription()) {
+				$documentation = 'https://portal.nextcloud.com/article/Nextcloud-Talk/High-Performance-Backend/Installation-of-Nextcloud-Talk-High-Performance-Backend';
+			}
+
 			return $setupResult(
 				$this->l->t('No High-performance backend configured - Running Nextcloud Talk without the High-performance backend only scales for very small calls (max. 2-3 participants). Please set up the High-performance backend to ensure calls with multiple participants work seamlessly.'),
-				'https://portal.nextcloud.com/article/Nextcloud-Talk/High-Performance-Backend/Installation-of-Nextcloud-Talk-High-Performance-Backend',
+				$documentation,
 			);
 		}
 

lib/SetupCheck/RecordingBackend.php

@@ -25,16 +25,20 @@ public function getCategory(): string {
 	}
 
 	public function getName(): string {
-		return $this->l->t('Recording backend');
+		$name = $this->l->t('Recording backend');
+		if ($this->talkConfig->getSignalingMode() === Config::SIGNALING_INTERNAL) {
+			return '[skip] ' . $name;
+		}
+		return $name;
 	}
 
 	public function run(): SetupResult {
-		if ($this->talkConfig->getSignalingMode() !== Config::SIGNALING_INTERNAL) {
-			return SetupResult::success();
+		if ($this->talkConfig->getSignalingMode() === Config::SIGNALING_INTERNAL) {
+			return SetupResult::success($this->l->t('Using the recording backend requires a High-performance backend.'));
 		}
 		if (empty($this->talkConfig->getRecordingServers())) {
 			return SetupResult::info($this->l->t('No recording backend configured'));
 		}
-		return SetupResult::error($this->l->t('Using the recording backend requires a High-performance backend.'));
+		return SetupResult::success();
 	}
 }

lib/SetupCheck/SIPConfiguration.php

@@ -25,16 +25,20 @@ public function getCategory(): string {
 	}
 
 	public function getName(): string {
-		return $this->l->t('SIP dial-in');
+		$name = $this->l->t('SIP configuration');
+		if ($this->talkConfig->getSignalingMode() === Config::SIGNALING_INTERNAL) {
+			return '[skip] ' . $name;
+		}
+		return $name;
 	}
 
 	public function run(): SetupResult {
-		if ($this->talkConfig->getSignalingMode() !== Config::SIGNALING_INTERNAL) {
-			return SetupResult::success();
+		if ($this->talkConfig->getSignalingMode() === Config::SIGNALING_INTERNAL) {
+			return SetupResult::success($this->l->t('Using the SIP functionality requires a High-performance backend.'));
 		}
 		if ($this->talkConfig->getSIPSharedSecret() === '' && $this->talkConfig->getDialInInfo() === '') {
 			return SetupResult::info($this->l->t('No SIP backend configured'));
 		}
-		return SetupResult::error($this->l->t('Using the SIP functionality requires a High-performance backend.'));
+		return SetupResult::success();
 	}
 }

src/components/AdminSettings/SignalingServers.vue

@@ -6,9 +6,24 @@
 <template>
 	<section id="signaling_server" class="signaling-servers section">
 		<NcNoteCard v-if="!servers.length"
-			type="error"
-			:heading="t('spreed', 'Nextcloud Talk setup not complete')"
-			:text="t('spreed', 'Install the High-performance backend to ensure calls with multiple participants work seamlessly.')" />
+			type="warning"
+			:heading="t('spreed', 'Nextcloud Talk setup not complete')">
+			{{ t('spreed', 'Please note that in calls with more than 2 participants without the High-performance backend, participants will most likely experience connectivity issues and cause high load on participating devices.') }}
+			{{ t('spreed', 'Install the High-performance backend to ensure calls with multiple participants work seamlessly.') }}
+
+			<NcButton v-if="hasValidSubscription"
+				type="primary"
+				class="additional-top-margin"
+				href="https://portal.nextcloud.com/article/Nextcloud-Talk/High-Performance-Backend/Installation-of-Nextcloud-Talk-High-Performance-Backend">
+				{{ t('spreed', 'Nextcloud portal') }} ↗
+			</NcButton>
+			<NcButton v-else
+				type="primary"
+				class="additional-top-margin"
+				href="https://nextcloud-talk.readthedocs.io/en/latest/quick-install/">
+				{{ t('spreed', 'Quick installation guide') }} ↗
+			</NcButton>
+		</NcNoteCard>
 
 		<h2>
 			{{ t('spreed', 'High-performance backend') }}
@@ -60,10 +75,9 @@
 			@update:model-value="debounceUpdateServers" />
 
 		<template v-if="!servers.length">
-			<NcNoteCard type="warning"
-				class="additional-top-margin"
-				:text="t('spreed', 'Please note that in calls with more than 2 participants without the High-performance backend, participants will most likely experience connectivity issues and cause high load on participating devices.')" />
 			<NcCheckboxRadioSwitch v-model="hideWarning"
+				type="switch"
+				class="additional-top-margin"
 				:disabled="loading"
 				@update:model-value="updateHideWarning">
 				{{ t('spreed', 'Don\'t warn about connectivity issues in calls with more than 2 participants') }}
@@ -112,6 +126,7 @@ export default {
 			hideWarning: false,
 			loading: false,
 			saved: false,
+			hasValidSubscription: loadState('spreed', 'has_valid_subscription'),
 			isCacheConfigured: loadState('spreed', 'has_cache_configured'),
 			isClusteredMode: loadState('spreed', 'signaling_mode') === SIGNALING.MODE.CLUSTER_CONVERSATION,
 			debounceUpdateServers: () => {},
@@ -149,7 +164,9 @@ export default {
 
 			OCP.AppConfig.setValue('spreed', 'hide_signaling_warning', this.hideWarning ? 'yes' : 'no', {
 				success: () => {
-					showSuccess(t('spreed', 'Missing High-performance backend warning hidden'))
+					if (this.hideWarning) {
+						showSuccess(t('spreed', 'Missing High-performance backend warning hidden'))
+					}
 					this.loading = false
 					this.toggleSave()
 				},
@@ -192,6 +209,6 @@ export default {
 }
 
 .additional-top-margin {
-	margin-top: 35px !important;
+	margin-top: 1em !important;
 }
 </style>