[PR #2201] [CLOSED] Revert "[feat] Create a "view" for all database references in a document" #5147

New issue

Closed

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

mirror commented

2026-03-23 22:17:27 +00:00

Owner

📋 Pull Request Information

Original PR: https://github.com/AppFlowy-IO/AppFlowy/pull/2201
Author: @a-wallen
Created: 4/5/2023
Status: ❌ Closed

Base: main ← Head: revert-2083-view_of_database

📝 Commits (1)

f7abb8b Revert "feat: Create a "view" for all database references in a document (#2083)"

📊 Changes

19 files changed (+401 additions, -714 deletions)

View changed files

📝 .github/workflows/integration_test.yml (+4 -3)
📝 .github/workflows/release.yml (+7 -23)
📝 frontend/.vscode/tasks.json (+222 -224)
📝 frontend/appflowy_flutter/.gitignore (+2 -2)
➖ frontend/appflowy_flutter/assets/test/workspaces/board.zip (+0 -0)
➖ frontend/appflowy_flutter/assets/test/workspaces/empty_document.zip (+0 -0)
📝 frontend/appflowy_flutter/assets/translations/en.json (+2 -4)
➖ frontend/appflowy_flutter/integration_test/board_test.dart (+0 -43)
➖ frontend/appflowy_flutter/integration_test/empty_document_test.dart (+0 -120)
➖ frontend/appflowy_flutter/integration_test/runner.dart (+0 -19)
📝 frontend/appflowy_flutter/integration_test/switch_folder_test.dart (+135 -0)
➖ frontend/appflowy_flutter/integration_test/util/data.dart (+0 -66)
➖ frontend/appflowy_flutter/integration_test/util/keyboard.dart (+0 -12)
📝 frontend/appflowy_flutter/integration_test/util/util.dart (+0 -1)
📝 frontend/appflowy_flutter/lib/plugins/database_view/application/database_service.dart (+1 -1)
📝 frontend/appflowy_flutter/lib/plugins/document/presentation/plugins/base/insert_page_command.dart (+2 -48)
📝 frontend/appflowy_flutter/pubspec.yaml (+26 -5)
➖ frontend/scripts/flutter_release_build/build_flowy.dart (+0 -28)
➖ frontend/scripts/flutter_release_build/tool.dart (+0 -115)

📄 Description

See https://github.com/AppFlowy-IO/AppFlowy/issues/2200. Integration tests seem to be failing due to a timeout exception. After revert, I will try to re-run integration tests to see whether this PR is the root cause.

Unfortunately, the integration tests seem to run fine on local machines, they just happen to fail on GitHub Actions.

_{🔄 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/2201 **Author:** [@a-wallen](https://github.com/a-wallen) **Created:** 4/5/2023 **Status:** ❌ Closed **Base:** `main` ← **Head:** `revert-2083-view_of_database` --- ### 📝 Commits (1) - [`f7abb8b`](https://github.com/AppFlowy-IO/AppFlowy/commit/f7abb8bfd71009a2140c59192b8597fbd47951af) Revert "feat: Create a "view" for all database references in a document (#2083)" ### 📊 Changes **19 files changed** (+401 additions, -714 deletions) <details> <summary>View changed files</summary> 📝 `.github/workflows/integration_test.yml` (+4 -3) 📝 `.github/workflows/release.yml` (+7 -23) 📝 `frontend/.vscode/tasks.json` (+222 -224) 📝 `frontend/appflowy_flutter/.gitignore` (+2 -2) ➖ `frontend/appflowy_flutter/assets/test/workspaces/board.zip` (+0 -0) ➖ `frontend/appflowy_flutter/assets/test/workspaces/empty_document.zip` (+0 -0) 📝 `frontend/appflowy_flutter/assets/translations/en.json` (+2 -4) ➖ `frontend/appflowy_flutter/integration_test/board_test.dart` (+0 -43) ➖ `frontend/appflowy_flutter/integration_test/empty_document_test.dart` (+0 -120) ➖ `frontend/appflowy_flutter/integration_test/runner.dart` (+0 -19) 📝 `frontend/appflowy_flutter/integration_test/switch_folder_test.dart` (+135 -0) ➖ `frontend/appflowy_flutter/integration_test/util/data.dart` (+0 -66) ➖ `frontend/appflowy_flutter/integration_test/util/keyboard.dart` (+0 -12) 📝 `frontend/appflowy_flutter/integration_test/util/util.dart` (+0 -1) 📝 `frontend/appflowy_flutter/lib/plugins/database_view/application/database_service.dart` (+1 -1) 📝 `frontend/appflowy_flutter/lib/plugins/document/presentation/plugins/base/insert_page_command.dart` (+2 -48) 📝 `frontend/appflowy_flutter/pubspec.yaml` (+26 -5) ➖ `frontend/scripts/flutter_release_build/build_flowy.dart` (+0 -28) ➖ `frontend/scripts/flutter_release_build/tool.dart` (+0 -115) </details> ### 📄 Description See https://github.com/AppFlowy-IO/AppFlowy/issues/2200. Integration tests seem to be failing due to a timeout exception. After revert, I will try to re-run integration tests to see whether this PR is the root cause. Unfortunately, the integration tests seem to run fine on local machines, they just happen to fail on GitHub Actions. --- <sub>🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.</sub>