[PR #2162] [MERGED] Fix the app's Dockerfile and comment it #5133

Closed
opened 2026-03-23 22:17:23 +00:00 by mirror · 0 comments
Owner

📋 Pull Request Information

Original PR: https://github.com/AppFlowy-IO/AppFlowy/pull/2162
Author: @resolritter
Created: 3/31/2023
Status: Merged
Merged: 4/11/2023
Merged by: @LucasXu0

Base: mainHead: docker


📝 Commits (2)

  • f22f356 chore: fix the app's Dockerfile and comment it
  • b4b69ed chore: test docker-compose on CI

📊 Changes

4 files changed (+121 additions, -46 deletions)

View changed files

📝 .dockerignore (+1 -4)
.github/workflows/docker_ci.yml (+47 -0)
📝 frontend/scripts/docker-buildfiles/Dockerfile (+62 -35)
📝 frontend/scripts/docker-buildfiles/docker-compose.yml (+11 -7)

📄 Description

Description

While trying to run the app through Docker (following https://appflowy.gitbook.io/docs/essential-documentation/install-appflowy/installation-methods/installing-with-docker) I ran into the following issues:

While going through those issues and solving them, I tinkered with the image and added some comments along the way.

Trying it out

$ cd frontend/scripts/docker-buildfiles
$ docker-compose rm -fvs
$ docker image rm --force appflowy/appflowy:latest
$ docker-compose up

🔄 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/AppFlowy-IO/AppFlowy/pull/2162 **Author:** [@resolritter](https://github.com/resolritter) **Created:** 3/31/2023 **Status:** ✅ Merged **Merged:** 4/11/2023 **Merged by:** [@LucasXu0](https://github.com/LucasXu0) **Base:** `main` ← **Head:** `docker` --- ### 📝 Commits (2) - [`f22f356`](https://github.com/AppFlowy-IO/AppFlowy/commit/f22f356db2639a718142eb9148f0caea16b9247e) chore: fix the app's Dockerfile and comment it - [`b4b69ed`](https://github.com/AppFlowy-IO/AppFlowy/commit/b4b69ede0089de2d74976330babe1b8a53fa5b7c) chore: test docker-compose on CI ### 📊 Changes **4 files changed** (+121 additions, -46 deletions) <details> <summary>View changed files</summary> 📝 `.dockerignore` (+1 -4) ➕ `.github/workflows/docker_ci.yml` (+47 -0) 📝 `frontend/scripts/docker-buildfiles/Dockerfile` (+62 -35) 📝 `frontend/scripts/docker-buildfiles/docker-compose.yml` (+11 -7) </details> ### 📄 Description # Description While trying to run the app through Docker (following https://appflowy.gitbook.io/docs/essential-documentation/install-appflowy/installation-methods/installing-with-docker) I ran into the following issues: - [The app is only compatible with Flutter 3.3.10](https://github.com/AppFlowy-IO/AppFlowy/issues/1741), but the Flutter version is not pinned in the Docker image. Solved by downloading the precise version according to https://docs.flutter.dev/get-started/install/linux#install-flutter-manually. - I needed to run `xhost local:docker` in my host prior to starting the app, otherwise an error will occur. Solved by running the aforementioned command and adding a note about it. - The entrypoint `["./appflowy_flutter"]` apparently is incorrect Solved by figuring out where the executable is located within the generated bundle with `find . -executable -type f` While going through those issues and solving them, I tinkered with the image and added some comments along the way. # Trying it out ```console $ cd frontend/scripts/docker-buildfiles $ docker-compose rm -fvs $ docker image rm --force appflowy/appflowy:latest $ docker-compose up ``` --- <sub>🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.</sub>
mirror 2026-03-23 22:17:23 +00:00
Sign in to join this conversation.
No milestone
No project
No assignees
1 participant
Notifications
Due date
The due date is invalid or out of range. Please use the format "yyyy-mm-dd".

No due date set.

Dependencies

No dependencies set.

Reference
AppFlowy-IO/AppFlowy#5133
No description provided.