[PR #2] [MERGED] Add fix for unclosed HTML breaks #61

New issue

Closed

opened 2026-03-23 20:35:36 +00:00 by mirror · 0 comments

mirror commented

2026-03-23 20:35:36 +00:00

Owner

📋 Pull Request Information

Original PR: https://github.com/tubearchivist/tubearchivist-jf-plugin/pull/2
Author: @ThisIsHetz
Created: 4/18/2024
Status: ✅ Merged
Merged: 4/18/2024
Merged by: @DarkFighterLuke

Base: master ← Head: ThisIsHetz-Patch-1

📝 Commits (1)

30715d5 Add fix for unclosed HTML breaks

📊 Changes

1 file changed (+2 additions, -1 deletions)

View changed files

📝 Jellyfin.Plugin.TubeArchivistMetadata/Utils/Utils.cs (+2 -1)

📄 Description

This changes the order of operations, truncating description before replacing newlines with HTML breaks, to prevent the creation of unclosed breaks which may otherwise cause rendering issues in Jellyfin.

Edit: I extend my apologies if it is considered poor etiquette to submit this pull request before you've had a chance to reply to the relevant issue. I've been taking this as an opportunity to better familiarize myself with git and GitHub, both of which I'm fairly naive to.

_{🔄 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/tubearchivist/tubearchivist-jf-plugin/pull/2 **Author:** [@ThisIsHetz](https://github.com/ThisIsHetz) **Created:** 4/18/2024 **Status:** ✅ Merged **Merged:** 4/18/2024 **Merged by:** [@DarkFighterLuke](https://github.com/DarkFighterLuke) **Base:** `master` ← **Head:** `ThisIsHetz-Patch-1` --- ### 📝 Commits (1) - [`30715d5`](https://github.com/tubearchivist/tubearchivist-jf-plugin/commit/30715d5c26e0dd9db1fc0cba97706bd80fdddd9b) Add fix for unclosed HTML breaks ### 📊 Changes **1 file changed** (+2 additions, -1 deletions) <details> <summary>View changed files</summary> 📝 `Jellyfin.Plugin.TubeArchivistMetadata/Utils/Utils.cs` (+2 -1) </details> ### 📄 Description This changes the order of operations, truncating description before replacing newlines with HTML breaks, to prevent the creation of unclosed breaks which may otherwise cause rendering issues in Jellyfin. Edit: I extend my apologies if it is considered poor etiquette to submit this pull request before you've had a chance to reply to the relevant issue. I've been taking this as an opportunity to better familiarize myself with git and GitHub, both of which I'm fairly naive to. --- <sub>🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.</sub>