[PR #2629] [CLOSED] Configure OLLAMA_ORIGINS via settings.json #57647

Closed
opened 2026-04-29 12:20:21 -05:00 by GiteaMirror · 0 comments
Owner

📋 Pull Request Information

Original PR: https://github.com/ollama/ollama/pull/2629
Author: @lovincyrus
Created: 2/21/2024
Status: Closed

Base: mainHead: cyrus/settings-json


📝 Commits (6)

  • 219f870 open settings.json in electron
  • 4d414b8 initial pass
  • 8629f37 get dynamic settings path from os, use origins
  • cf28702 clean up unused
  • f5fc09c support dynamic settings path in updateTray
  • c73f7b3 move settings.json to root instead of models

📊 Changes

2 files changed (+148 additions, -36 deletions)

View changed files

📝 macapp/src/index.ts (+43 -9)
📝 server/routes.go (+105 -27)

📄 Description

Took a stab at these issues https://github.com/ollama/ollama/issues/2335, https://github.com/ollama/ollama/issues/2369

Added settings menu item in the Electron tray application. Also, hoisted the OLLAMA_ORIGINS environment variable to the settings.json file, ensuring routes.go retrieves origins from the file rather than the environment variable.

Open for suggestions.

CleanShot 2024-02-21 at 18 09 37@2x

CleanShot 2024-02-21 at 18 10 05@2x


🔄 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/ollama/ollama/pull/2629 **Author:** [@lovincyrus](https://github.com/lovincyrus) **Created:** 2/21/2024 **Status:** ❌ Closed **Base:** `main` ← **Head:** `cyrus/settings-json` --- ### 📝 Commits (6) - [`219f870`](https://github.com/ollama/ollama/commit/219f870082c8e262206fde83510a0a82302bd496) open settings.json in electron - [`4d414b8`](https://github.com/ollama/ollama/commit/4d414b86b585fe3e6686c35e9608a860676f62a8) initial pass - [`8629f37`](https://github.com/ollama/ollama/commit/8629f37205f2b5debc48a3a983d73b2dbf5e64ae) get dynamic settings path from os, use origins - [`cf28702`](https://github.com/ollama/ollama/commit/cf28702265da79e73d305f94e2eaf10af9f81f01) clean up unused - [`f5fc09c`](https://github.com/ollama/ollama/commit/f5fc09c1684d2576d6ff3d2b35285b5c010bd7ae) support dynamic settings path in updateTray - [`c73f7b3`](https://github.com/ollama/ollama/commit/c73f7b32e66969735198a3b07b0043f2c7282110) move settings.json to root instead of models ### 📊 Changes **2 files changed** (+148 additions, -36 deletions) <details> <summary>View changed files</summary> 📝 `macapp/src/index.ts` (+43 -9) 📝 `server/routes.go` (+105 -27) </details> ### 📄 Description Took a stab at these issues https://github.com/ollama/ollama/issues/2335, https://github.com/ollama/ollama/issues/2369 Added settings menu item in the Electron tray application. Also, hoisted the `OLLAMA_ORIGINS` environment variable to the settings.json file, ensuring routes.go retrieves origins from the file rather than the environment variable. Open for suggestions. ![CleanShot 2024-02-21 at 18 09 37@2x](https://github.com/ollama/ollama/assets/1021101/61a2ba18-2b52-4d53-9eed-38c2c3d87f89) ![CleanShot 2024-02-21 at 18 10 05@2x](https://github.com/ollama/ollama/assets/1021101/df9bb6a2-7246-40af-9b98-f5e066d427c9) --- <sub>🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.</sub>
GiteaMirror added the pull-request label 2026-04-29 12:20:21 -05:00
Sign in to join this conversation.
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: github-starred/ollama#57647