Skip to content

Webui overhaul - #125

Merged
coffandro merged 140 commits into
mainfrom
webui_overhaul
May 23, 2026
Merged

Webui overhaul#125
coffandro merged 140 commits into
mainfrom
webui_overhaul

Conversation

@coffandro

@coffandro coffandro commented Jan 8, 2026

Copy link
Copy Markdown
Collaborator

Fixes: #79

@edvardm

edvardm commented Apr 13, 2026

Copy link
Copy Markdown
Contributor

Would it be ok to allow plus chars in filename? eg. pla+ would be thus a valid substring in a file path

@coffandro

Copy link
Copy Markdown
Collaborator Author

That does not seem to be allowed, no
Does the new Webui do this?

@edvardm

edvardm commented Apr 13, 2026

Copy link
Copy Markdown
Contributor

I mean upstream seems to allow it? but maybe even better question is: how it should be? I mean this line in main does accept + intentionally, it seems

@coffandro
coffandro requested a review from Copilot May 23, 2026 11:41

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Copilot wasn't able to review this pull request because it exceeds the maximum number of files (300). Try reducing the number of changed files and requesting a review from Copilot again.

@coffandro

Copy link
Copy Markdown
Collaborator Author

Preparing to merge into main...
After many months of development

@coffandro
coffandro merged commit ec3b986 into main May 23, 2026
6 checks passed
@coffandro
coffandro deleted the webui_overhaul branch May 23, 2026 13:55
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

enhancement New feature or request webui

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[Feature Request] rework webui into cloud deployable webapp.

5 participants