[PR #573] [MERGED] Sanitize all playlist descriptions #603

Closed
opened 2026-02-28 14:33:36 +03:00 by kerem · 0 comments
Owner

📋 Pull Request Information

Original PR: https://github.com/jpochyla/psst/pull/573
Author: @jacksongoode
Created: 12/24/2024
Status: Merged
Merged: 12/28/2024
Merged by: @jacksongoode

Base: mainHead: jackson/sanitize-descriptions


📝 Commits (4)

📊 Changes

2 files changed (+38 additions, -26 deletions)

View changed files

📝 psst-gui/src/data/playlist.rs (+12 -0)
📝 psst-gui/src/webapi/client.rs (+26 -26)

📄 Description

This should in theory cover all descriptions from playlists and sanitize them appropriately. There were still issues with some descriptions that were in the search results, and ampersands were not properly replaced with their actual character.


🔄 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/jpochyla/psst/pull/573 **Author:** [@jacksongoode](https://github.com/jacksongoode) **Created:** 12/24/2024 **Status:** ✅ Merged **Merged:** 12/28/2024 **Merged by:** [@jacksongoode](https://github.com/jacksongoode) **Base:** `main` ← **Head:** `jackson/sanitize-descriptions` --- ### 📝 Commits (4) - [`708e9e3`](https://github.com/jpochyla/psst/commit/708e9e3314cd31ab800f688c997ff04a9099a897) Sanitize all playlist descriptions - [`695d60c`](https://github.com/jpochyla/psst/commit/695d60c141e96873775250c280c85dd2e835b9b6) Better name - [`1f9bdda`](https://github.com/jpochyla/psst/commit/1f9bdda09218362f3a63041f488d309ee568c470) Deserialize instead - [`ceb15bc`](https://github.com/jpochyla/psst/commit/ceb15bc9797d415777626128d5fca28b8b065ab1) Cleanup ### 📊 Changes **2 files changed** (+38 additions, -26 deletions) <details> <summary>View changed files</summary> 📝 `psst-gui/src/data/playlist.rs` (+12 -0) 📝 `psst-gui/src/webapi/client.rs` (+26 -26) </details> ### 📄 Description This should in theory cover all descriptions from playlists and sanitize them appropriately. There were still issues with some descriptions that were in the search results, and ampersands were not properly replaced with their actual character. --- <sub>🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.</sub>
kerem 2026-02-28 14:33:36 +03:00
Sign in to join this conversation.
No milestone
No project
No assignees
1 participant
Notifications
Due date
The due date is invalid or out of range. Please use the format "yyyy-mm-dd".

No due date set.

Dependencies

No dependencies set.

Reference
starred/psst#603
No description provided.