mirror of
https://github.com/open-webui/open-webui.git
synced 2026-05-11 08:22:09 -05:00
[PR #13832] [MERGED] feat: Handle frontend updates according to Svelte docs #23303
Reference in New Issue
Block a user
Delete Branch "%!s()"
Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?
📋 Pull Request Information
Original PR: https://github.com/open-webui/open-webui/pull/13832
Author: @Ithanil
Created: 5/13/2025
Status: ✅ Merged
Merged: 5/14/2025
Merged by: @tjbck
Base:
dev← Head:svelte_versioning📝 Commits (1)
dfd4037better handling of frontend updates, following svelte docs📊 Changes
3 files changed (+20 additions, -1 deletions)
View changed files
📝
Dockerfile(+3 -0)📝
src/routes/+layout.svelte(+10 -0)📝
svelte.config.js(+7 -1)📄 Description
Pull Request Checklist
Note to first-time contributors: Please open a discussion post in Discussions and describe your changes before submitting a pull request.
Before submitting, make sure you've checked the following:
devbranch.Changelog Entry
Description
Disclaimer: I don't really know stuff about Svelte at all, just did some research and followed their documentation.
Discussions like https://github.com/open-webui/open-webui/discussions/13586 or the comment https://github.com/open-webui/open-webui/pull/13364#issuecomment-2871939637 have led me to research what could be improved in terms of handling frontend code updates. I ended up with this section in the Svelte docs: https://svelte.dev/docs/kit/configuration#version
The present PR implements the suggestions from there, using a polling interval of 60s (just a tiny request). The "version" of the app is the commit hash it was built from.
In testing, I have verified the polling and updated the container to a new revision with a tab open, which led to a full page reload at least when "clicking" something, e.g. opening workspaces.
Added
Screenshot
Additional Information
Note: I still think the modifications for index.html caching should be done additionally.
Contributor License Agreement
By submitting this pull request, I confirm that I have read and fully agree to the Contributor License Agreement (CLA), and I am providing my contributions under its terms.
🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.