[PR #2932] [MERGED] Add optional camelCase formatter #9280

Closed
opened 2026-02-07 05:59:22 +03:00 by OVERLORD · 0 comments
Owner

📋 Pull Request Information

Original PR: https://github.com/jellyfin/jellyfin/pull/2932
Author: @crobibero
Created: 4/20/2020
Status: Merged
Merged: 4/25/2020
Merged by: @Bond-009

Base: api-migrationHead: api-casing


📝 Commits (4)

  • ad67081 add camelCase formatter
  • c89dc89 Fix PascalCase
  • fe63214 Move Json Options to static class for easier access.
  • 0765ef8 Apply suggestions, fix warning

📊 Changes

4 files changed (+88 additions, -0 deletions)

View changed files

📝 Jellyfin.Server/Extensions/ApiServiceCollectionExtensions.cs (+3 -0)
Jellyfin.Server/Formatters/CamelCaseJsonProfileFormatter.cs (+21 -0)
Jellyfin.Server/Formatters/PascalCaseJsonProfileFormatter.cs (+23 -0)
Jellyfin.Server/Models/JsonOptions.cs (+41 -0)

📄 Description

With Accept: application/json;profile="CamelCase"
Response:

{
  "key1": "Value 1",
  "key2": "vAlUe 2"
}

With Accept: application/json or Accept: application/json;profile="PascalCase"
Response:

{
  "Key1": "Value 1",
  "Key2": "vAlUe 2"
}

Issues
Fixes #2930


🔄 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/2932 **Author:** [@crobibero](https://github.com/crobibero) **Created:** 4/20/2020 **Status:** ✅ Merged **Merged:** 4/25/2020 **Merged by:** [@Bond-009](https://github.com/Bond-009) **Base:** `api-migration` ← **Head:** `api-casing` --- ### 📝 Commits (4) - [`ad67081`](https://github.com/jellyfin/jellyfin/commit/ad67081840ec61085673634795d0b6363f649dbf) add camelCase formatter - [`c89dc89`](https://github.com/jellyfin/jellyfin/commit/c89dc8921ffb0ce11031e9cfb096b525d94e21b3) Fix PascalCase - [`fe63214`](https://github.com/jellyfin/jellyfin/commit/fe632146dcba69edeec56b850736227ff5f4c5b3) Move Json Options to static class for easier access. - [`0765ef8`](https://github.com/jellyfin/jellyfin/commit/0765ef8bda4d23e33fde7a1bfe49b5a365c6d28e) Apply suggestions, fix warning ### 📊 Changes **4 files changed** (+88 additions, -0 deletions) <details> <summary>View changed files</summary> 📝 `Jellyfin.Server/Extensions/ApiServiceCollectionExtensions.cs` (+3 -0) ➕ `Jellyfin.Server/Formatters/CamelCaseJsonProfileFormatter.cs` (+21 -0) ➕ `Jellyfin.Server/Formatters/PascalCaseJsonProfileFormatter.cs` (+23 -0) ➕ `Jellyfin.Server/Models/JsonOptions.cs` (+41 -0) </details> ### 📄 Description With `Accept: application/json;profile="CamelCase"` Response: ``` { "key1": "Value 1", "key2": "vAlUe 2" } ``` With `Accept: application/json` or `Accept: application/json;profile="PascalCase"` Response: ``` { "Key1": "Value 1", "Key2": "vAlUe 2" } ``` **Issues** Fixes #2930 --- <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 05:59:22 +03:00
Sign in to join this conversation.
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: starred/jellyfin#9280