fix: notifications link for mobile / desktop

Fixes #316 Signed-off-by: Josh <josh.t.richards@gmail.com>
nextcloud · Sep 24, 2024 · 70bc6a4 · 70bc6a4
1 parent ba4dac1
commit 70bc6a4
Showing 1 changed file with 4 additions and 2 deletions.
diff --git a/lib/BackgroundJobs/AdminNotification.php b/lib/BackgroundJobs/AdminNotification.php
@@ -36,15 +36,17 @@ protected function run($argument): void {
 			->setSubject('updated')
 			->setObject('dummy', '23');
 
+		$enableLink = $this->url->getAbsoluteURL($this->url->linkTo('', 'ocs/v2.php/apps/survey_client/api/v1/monthly'), 'POST'));
 		$enableAction = $notification->createAction();
 		$enableAction->setLabel('enable')
-			->setLink($this->url->getAbsoluteURL('ocs/v2.php/apps/survey_client/api/v1/monthly'), 'POST')
+			->setLink($enableLink)
 			->setPrimary(true);
 		$notification->addAction($enableAction);
 
+		$disableLink = $this->url->getAbsoluteURL($this->url->linkTo('', 'ocs/v2.php/apps/survey_client/api/v1/monthly'), 'DELETE'));
 		$disableAction = $notification->createAction();
 		$disableAction->setLabel('disable')
-			->setLink($this->url->getAbsoluteURL('ocs/v2.php/apps/survey_client/api/v1/monthly'), 'DELETE')
+			->setLink($disableLink)
 			->setPrimary(false);
 		$notification->addAction($disableAction);