Skip to content

Commit

Permalink
Merge pull request #322 from nextcloud/backport/317/stable28
Browse files Browse the repository at this point in the history
[stable28] fix: notifications link for mobile / desktop error generating errors in v30
  • Loading branch information
nickvergessen authored Sep 26, 2024
2 parents 95a871d + a763c3d commit 5125d88
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 @@ -50,15 +50,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 5125d88

Please sign in to comment.