[PR #3466] [MERGED] Make all optional strings nullable #9544

Closed
opened 2026-02-07 06:04:07 +03:00 by OVERLORD · 0 comments
Owner

📋 Pull Request Information

Original PR: https://github.com/jellyfin/jellyfin/pull/3466
Author: @crobibero
Created: 6/27/2020
Status: Merged
Merged: 6/29/2020
Merged by: @barronpm

Base: api-migrationHead: api-migration-nullable


📝 Commits (1)

  • 73bcda7 Make all optional strings nullable

📊 Changes

33 files changed (+143 additions, -145 deletions)

View changed files

📝 Jellyfin.Api/Controllers/AlbumsController.cs (+2 -2)
📝 Jellyfin.Api/Controllers/ApiKeyController.cs (+2 -2)
📝 Jellyfin.Api/Controllers/CollectionController.cs (+4 -4)
📝 Jellyfin.Api/Controllers/ConfigurationController.cs (+2 -2)
📝 Jellyfin.Api/Controllers/DashboardController.cs (+1 -1)
📝 Jellyfin.Api/Controllers/DevicesController.cs (+4 -4)
📝 Jellyfin.Api/Controllers/DisplayPreferencesController.cs (+6 -6)
📝 Jellyfin.Api/Controllers/ImageByNameController.cs (+6 -6)
📝 Jellyfin.Api/Controllers/InstantMixController.cs (+8 -8)
📝 Jellyfin.Api/Controllers/ItemUpdateController.cs (+1 -1)
📝 Jellyfin.Api/Controllers/LibraryController.cs (+8 -8)
📝 Jellyfin.Api/Controllers/LibraryStructureController.cs (+11 -11)
📝 Jellyfin.Api/Controllers/NotificationsController.cs (+2 -2)
📝 Jellyfin.Api/Controllers/PackageController.cs (+4 -4)
📝 Jellyfin.Api/Controllers/PlaylistsController.cs (+7 -7)
📝 Jellyfin.Api/Controllers/PluginsController.cs (+2 -2)
📝 Jellyfin.Api/Controllers/RemoteImageController.cs (+1 -1)
📝 Jellyfin.Api/Controllers/ScheduledTasksController.cs (+4 -4)
📝 Jellyfin.Api/Controllers/SearchController.cs (+5 -5)
📝 Jellyfin.Api/Controllers/SessionController.cs (+23 -23)

...and 13 more files

📄 Description

Optional query parameters must be nullable.


🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.

## 📋 Pull Request Information **Original PR:** https://github.com/jellyfin/jellyfin/pull/3466 **Author:** [@crobibero](https://github.com/crobibero) **Created:** 6/27/2020 **Status:** ✅ Merged **Merged:** 6/29/2020 **Merged by:** [@barronpm](https://github.com/barronpm) **Base:** `api-migration` ← **Head:** `api-migration-nullable` --- ### 📝 Commits (1) - [`73bcda7`](https://github.com/jellyfin/jellyfin/commit/73bcda7eac6d0785745179fe4b7f58b6bc4ec488) Make all optional strings nullable ### 📊 Changes **33 files changed** (+143 additions, -145 deletions) <details> <summary>View changed files</summary> 📝 `Jellyfin.Api/Controllers/AlbumsController.cs` (+2 -2) 📝 `Jellyfin.Api/Controllers/ApiKeyController.cs` (+2 -2) 📝 `Jellyfin.Api/Controllers/CollectionController.cs` (+4 -4) 📝 `Jellyfin.Api/Controllers/ConfigurationController.cs` (+2 -2) 📝 `Jellyfin.Api/Controllers/DashboardController.cs` (+1 -1) 📝 `Jellyfin.Api/Controllers/DevicesController.cs` (+4 -4) 📝 `Jellyfin.Api/Controllers/DisplayPreferencesController.cs` (+6 -6) 📝 `Jellyfin.Api/Controllers/ImageByNameController.cs` (+6 -6) 📝 `Jellyfin.Api/Controllers/InstantMixController.cs` (+8 -8) 📝 `Jellyfin.Api/Controllers/ItemUpdateController.cs` (+1 -1) 📝 `Jellyfin.Api/Controllers/LibraryController.cs` (+8 -8) 📝 `Jellyfin.Api/Controllers/LibraryStructureController.cs` (+11 -11) 📝 `Jellyfin.Api/Controllers/NotificationsController.cs` (+2 -2) 📝 `Jellyfin.Api/Controllers/PackageController.cs` (+4 -4) 📝 `Jellyfin.Api/Controllers/PlaylistsController.cs` (+7 -7) 📝 `Jellyfin.Api/Controllers/PluginsController.cs` (+2 -2) 📝 `Jellyfin.Api/Controllers/RemoteImageController.cs` (+1 -1) 📝 `Jellyfin.Api/Controllers/ScheduledTasksController.cs` (+4 -4) 📝 `Jellyfin.Api/Controllers/SearchController.cs` (+5 -5) 📝 `Jellyfin.Api/Controllers/SessionController.cs` (+23 -23) _...and 13 more files_ </details> ### 📄 Description Optional query parameters must be nullable. --- <sub>🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.</sub>
OVERLORD added the pull-request label 2026-02-07 06:04:07 +03:00
Sign in to join this conversation.
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: starred/jellyfin#9544