diff --git a/Jellyfin.Api/Controllers/NotificationsController.cs b/Jellyfin.Api/Controllers/NotificationsController.cs
index d9a5c5e316..76b025fa16 100644
--- a/Jellyfin.Api/Controllers/NotificationsController.cs
+++ b/Jellyfin.Api/Controllers/NotificationsController.cs
@@ -10,6 +10,7 @@ using MediaBrowser.Controller.Library;
using MediaBrowser.Controller.Notifications;
using MediaBrowser.Model.Dto;
using MediaBrowser.Model.Notifications;
+using Microsoft.AspNetCore.Http;
using Microsoft.AspNetCore.Mvc;
namespace Jellyfin.Api.Controllers
@@ -42,13 +43,14 @@ namespace Jellyfin.Api.Controllers
/// An optional limit on the number of notifications returned.
/// A read-only list of all of the user's notifications.
[HttpGet("{UserID}")]
- public NotificationResultDto GetNotifications(
+ [ProducesResponseType(typeof(IEnumerable), StatusCodes.Status200OK)]
+ public IActionResult GetNotifications(
[FromRoute] string userId,
[FromQuery] bool? isRead,
[FromQuery] int? startIndex,
[FromQuery] int? limit)
{
- return new NotificationResultDto();
+ return Ok(new NotificationResultDto());
}
///
@@ -57,10 +59,11 @@ namespace Jellyfin.Api.Controllers
/// The user's ID.
/// Notifications summary for the user.
[HttpGet("{UserID}/Summary")]
- public NotificationsSummaryDto GetNotificationsSummary(
+ [ProducesResponseType(typeof(NotificationsSummaryDto), StatusCodes.Status200OK)]
+ public IActionResult GetNotificationsSummary(
[FromRoute] string userId)
{
- return new NotificationsSummaryDto();
+ return Ok(new NotificationsSummaryDto());
}
///
@@ -68,9 +71,18 @@ namespace Jellyfin.Api.Controllers
///
/// All notification types.
[HttpGet("Types")]
- public IEnumerable GetNotificationTypes()
+ [ProducesResponseType(typeof(IEnumerable), StatusCodes.Status200OK)]
+ [ProducesResponseType(typeof(string), StatusCodes.Status500InternalServerError)]
+ public IActionResult GetNotificationTypes()
{
- return _notificationManager.GetNotificationTypes();
+ try
+ {
+ return Ok(_notificationManager.GetNotificationTypes());
+ }
+ catch (Exception e)
+ {
+ return StatusCode(StatusCodes.Status500InternalServerError, e.Message);
+ }
}
///
@@ -78,9 +90,18 @@ namespace Jellyfin.Api.Controllers
///
/// All notification services.
[HttpGet("Services")]
- public IEnumerable GetNotificationServices()
+ [ProducesResponseType(typeof(IEnumerable), StatusCodes.Status200OK)]
+ [ProducesResponseType(typeof(string), StatusCodes.Status500InternalServerError)]
+ public IActionResult GetNotificationServices()
{
- return _notificationManager.GetNotificationServices();
+ try
+ {
+ return Ok(_notificationManager.GetNotificationServices());
+ }
+ catch (Exception e)
+ {
+ return StatusCode(StatusCodes.Status500InternalServerError, e.Message);
+ }
}
///
@@ -90,24 +111,36 @@ namespace Jellyfin.Api.Controllers
/// The description of the notification.
/// The URL of the notification.
/// The level of the notification.
+ /// Status.
[HttpPost("Admin")]
- public void CreateAdminNotification(
+ [ProducesResponseType(StatusCodes.Status200OK)]
+ [ProducesResponseType(typeof(string), StatusCodes.Status500InternalServerError)]
+ public IActionResult CreateAdminNotification(
[FromQuery] string name,
[FromQuery] string description,
[FromQuery] string? url,
[FromQuery] NotificationLevel? level)
{
- var notification = new NotificationRequest
+ try
{
- Name = name,
- Description = description,
- Url = url,
- Level = level ?? NotificationLevel.Normal,
- UserIds = _userManager.Users.Where(i => i.Policy.IsAdministrator).Select(i => i.Id).ToArray(),
- Date = DateTime.UtcNow,
- };
+ var notification = new NotificationRequest
+ {
+ Name = name,
+ Description = description,
+ Url = url,
+ Level = level ?? NotificationLevel.Normal,
+ UserIds = _userManager.Users.Where(i => i.Policy.IsAdministrator).Select(i => i.Id).ToArray(),
+ Date = DateTime.UtcNow,
+ };
- _notificationManager.SendNotification(notification, CancellationToken.None);
+ _notificationManager.SendNotification(notification, CancellationToken.None);
+
+ return Ok();
+ }
+ catch (Exception e)
+ {
+ return StatusCode(StatusCodes.Status500InternalServerError, e.Message);
+ }
}
///
@@ -115,11 +148,14 @@ namespace Jellyfin.Api.Controllers
///
/// The userID.
/// A comma-separated list of the IDs of notifications which should be set as read.
+ /// Status.
[HttpPost("{UserID}/Read")]
- public void SetRead(
+ [ProducesResponseType(StatusCodes.Status200OK)]
+ public IActionResult SetRead(
[FromRoute] string userId,
[FromQuery] string ids)
{
+ return Ok();
}
///
@@ -127,11 +163,14 @@ namespace Jellyfin.Api.Controllers
///
/// The userID.
/// A comma-separated list of the IDs of notifications which should be set as unread.
+ /// Status.
[HttpPost("{UserID}/Unread")]
- public void SetUnread(
+ [ProducesResponseType(StatusCodes.Status200OK)]
+ public IActionResult SetUnread(
[FromRoute] string userId,
[FromQuery] string ids)
{
+ return Ok();
}
}
}