Skip to content

Commit

Permalink
Merge pull request #317 from nextcloud/jtr/fix-issue-316-mobile-url-n…
Browse files Browse the repository at this point in the history
…otifications

fix: notifications link for mobile / desktop error generating errors in v30
  • Loading branch information
nickvergessen authored Sep 26, 2024
2 parents bcd1417 + a984156 commit 5f4e8c3
Showing 1 changed file with 4 additions and 2 deletions.
6 changes: 4 additions & 2 deletions lib/BackgroundJobs/AdminNotification.php
Original file line number Diff line number Diff line change
Expand Up @@ -36,15 +36,17 @@ protected function run($argument): void {
->setSubject('updated')
->setObject('dummy', '23');

$enableLink = $this->url->linkToOCSRouteAbsolute('survey_client.Endpoint.enableMonthly');
$enableAction = $notification->createAction();
$enableAction->setLabel('enable')
->setLink($this->url->getAbsoluteURL('ocs/v2.php/apps/survey_client/api/v1/monthly'), 'POST')
->setLink($enableLink, 'POST')
->setPrimary(true);
$notification->addAction($enableAction);

$disableLink = $this->url->linkToOCSRouteAbsolute('survey_client.Endpoint.disableMonthly');
$disableAction = $notification->createAction();
$disableAction->setLabel('disable')
->setLink($this->url->getAbsoluteURL('ocs/v2.php/apps/survey_client/api/v1/monthly'), 'DELETE')
->setLink($disableLink, 'DELETE')
->setPrimary(false);
$notification->addAction($disableAction);

Expand Down

0 comments on commit 5f4e8c3

Please sign in to comment.