mirror of
https://github.com/open-webui/open-webui.git
synced 2026-07-15 21:19:39 -05:00
[PR #536] [MERGED] Unhardcode some stuff, flexibilize dotenv requirement #20364
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/536
Author: @lucasew
Created: 1/19/2024
Status: ✅ Merged
Merged: 1/25/2024
Merged by: @tjbck
Base:
main← Head:backend-data-dir📝 Commits (5)
e3503d6backend: make dotenv optional5b26d2abackend: make the data directory and the artifacts from the frontend customizable using environment variablesd2c5f3drefac: convert str var to f-string8da06f5fixes after the refactor326af5arefac📊 Changes
4 files changed (+30 additions, -23 deletions)
View changed files
📝
backend/apps/web/internal/db.py(+3 -1)📝
backend/apps/web/routers/utils.py(+3 -5)📝
backend/config.py(+18 -15)📝
backend/main.py(+6 -2)📄 Description
Today Docker and Systemd already have ways of mimicking what dotenv does, but for anything that runs under it. It's a detail.
I am trying to Nix it. This PR started as I being lazy to patch stuff up to work with my requisites lol.
Even though one can customize some extra values, like the data directory, the defaults are still the same.
I tested the backend using a virtualenv environment to see if it finds stuff and works. I need to test if the ollama integration properly works.
🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.