[PR #7] [MERGED] Fix bug that allows whitespace before/after filename #149

Closed
opened 2026-03-01 18:43:55 +03:00 by kerem · 0 comments
Owner

📋 Pull Request Information

Original PR: https://github.com/netbootxyz/webapp/pull/7
Author: @vangorra
Created: 4/25/2021
Status: Merged
Merged: 4/28/2021
Merged by: @thelamer

Base: masterHead: vangorra_whitespace_trim


📝 Commits (1)

  • 5aca750 Adding trim for filename.

📊 Changes

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

View changed files

📝 public/js/netbootxyz-web.js (+1 -1)

📄 Description

When copying an pasting filenames, the whitespace is preserved in the filename but not presented in the UI. Removing the whitespace from the filename upon file creation avoids extensive troubleshooting.


🔄 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/netbootxyz/webapp/pull/7 **Author:** [@vangorra](https://github.com/vangorra) **Created:** 4/25/2021 **Status:** ✅ Merged **Merged:** 4/28/2021 **Merged by:** [@thelamer](https://github.com/thelamer) **Base:** `master` ← **Head:** `vangorra_whitespace_trim` --- ### 📝 Commits (1) - [`5aca750`](https://github.com/netbootxyz/webapp/commit/5aca75069465c3bd55fa7987a0452b358c8b460c) Adding trim for filename. ### 📊 Changes **1 file changed** (+1 additions, -1 deletions) <details> <summary>View changed files</summary> 📝 `public/js/netbootxyz-web.js` (+1 -1) </details> ### 📄 Description When copying an pasting filenames, the whitespace is preserved in the filename but not presented in the UI. Removing the whitespace from the filename upon file creation avoids extensive troubleshooting. --- <sub>🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.</sub>
kerem 2026-03-01 18:43:55 +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/webapp#149
No description provided.