[PR #1547] [CLOSED] Switch from "docker-compose" to "docker compose" if necessary (Makefile) + Multi file Drag&Drop + minor things #7498

Closed
opened 2025-11-11 17:28:24 -06:00 by GiteaMirror · 0 comments
Owner

📋 Pull Request Information

Original PR: https://github.com/open-webui/open-webui/pull/1547
Author: @Entaigner
Created: 4/14/2024
Status: Closed

Base: devHead: main


📝 Commits (3)

  • 5e4b37a Switch from "docker-compose" to "docker compose" in Makefile
  • 5514a7d Changed two forEach loops with anonymous functions to a backwards for loop
  • 35b2d9a Enable multi file drag & drop + better error handling for image uploads

📊 Changes

4 files changed (+108 additions, -83 deletions)

View changed files

📝 Makefile (+14 -8)
📝 confirm_remove.sh (+6 -1)
📝 src/lib/components/chat/MessageInput.svelte (+79 -61)
📝 src/routes/(app)/c/[id]/+page.svelte (+9 -13)

📄 Description

  • Detect docker compose using the bash functions "command" and "which" which are at least POSIX compatible.
  • Improved a minor code detail in the handling of images in chat messages before they get sent to the server
  • Enabled multi file drag & drop
  • Improved error handling for images after drag & drop
  • I've also added a text to the Drag&Drop code. If someone would be able to tell me if i need to do anything about that, other then committing it, i'll be listening.

Pull Request Checklist

  • Description: Briefly describe the changes in this pull request.
  • Changelog: Ensure a changelog entry following the format of Keep a Changelog is added at the bottom of the PR description.
  • [NO] Documentation: Have you updated relevant documentation?
  • [NO] Dependencies: Are there any new dependencies? Have you updated the dependency versions in the documentation?

Description

Added "docker compose" capability to the Makefile


Changelog Entry

Added

  • Added the capability to chose between "docker-compose" and "docker compose" to the "Makefile"
  • Handling of multi file drag & drop
  • Improved error handling for images after drag & drop

Changed

  • Turned two .forEach() forward loops into one for() backwards loop

🔄 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/open-webui/open-webui/pull/1547 **Author:** [@Entaigner](https://github.com/Entaigner) **Created:** 4/14/2024 **Status:** ❌ Closed **Base:** `dev` ← **Head:** `main` --- ### 📝 Commits (3) - [`5e4b37a`](https://github.com/open-webui/open-webui/commit/5e4b37a741e768e82c75bbb8caa53a168d0e7708) Switch from "docker-compose" to "docker compose" in Makefile - [`5514a7d`](https://github.com/open-webui/open-webui/commit/5514a7dccf6c1fc3df0701e0ecbdb0f155506cb5) Changed two forEach loops with anonymous functions to a backwards for loop - [`35b2d9a`](https://github.com/open-webui/open-webui/commit/35b2d9ac0ec87bb5a4d20e2b3dede54c0b677154) Enable multi file drag & drop + better error handling for image uploads ### 📊 Changes **4 files changed** (+108 additions, -83 deletions) <details> <summary>View changed files</summary> 📝 `Makefile` (+14 -8) 📝 `confirm_remove.sh` (+6 -1) 📝 `src/lib/components/chat/MessageInput.svelte` (+79 -61) 📝 `src/routes/(app)/c/[id]/+page.svelte` (+9 -13) </details> ### 📄 Description - Detect docker compose using the bash functions "command" and "which" which are at least POSIX compatible. - Improved a minor code detail in the handling of images in chat messages before they get sent to the server - Enabled multi file drag & drop - Improved error handling for images after drag & drop - I've also added a text to the Drag&Drop code. If someone would be able to tell me if i need to do anything about that, other then committing it, i'll be listening. ## Pull Request Checklist - [X] **Description:** Briefly describe the changes in this pull request. - [X] **Changelog:** Ensure a changelog entry following the format of [Keep a Changelog](https://keepachangelog.com/) is added at the bottom of the PR description. - [NO] **Documentation:** Have you updated relevant documentation? - [NO] **Dependencies:** Are there any new dependencies? Have you updated the dependency versions in the documentation? --- ## Description Added "docker compose" capability to the Makefile --- ### Changelog Entry ### Added - Added the capability to chose between "docker-compose" and "docker compose" to the "Makefile" - Handling of multi file drag & drop - Improved error handling for images after drag & drop ### Changed - Turned two .forEach() forward loops into one for() backwards loop --- <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 2025-11-11 17:28:24 -06:00
Sign in to join this conversation.
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: github-starred/open-webui#7498