[PR #6130] feat: fetch url meta data on creating row #7210

New issue

Open

opened 2026-03-23 23:18:36 +00:00 by mirror · 0 comments

mirror commented

2026-03-23 23:18:36 +00:00

Owner

📋 Pull Request Information

Original PR: https://github.com/AppFlowy-IO/AppFlowy/pull/6130
Author: @zoli
Created: 8/30/2024
Status: 🔄 Open

Base: main ← Head: feat/fetch-url-meta-data

📝 Commits (6)

7057439 feat: fetch url meta data on creating row
ec9bcca style: merge main
43f4305 style: merge main
c1abc5d fix: url meta fetching state
5ed1724 feat: select url field to fill on fetching meta
347d0fc test: added integration and bloc test for fetching url meta data

📊 Changes

17 files changed (+504 additions, -34 deletions)

View changed files

📝 frontend/appflowy_flutter/integration_test/desktop/board/board_add_row_test.dart (+67 -0)
📝 frontend/appflowy_flutter/integration_test/shared/database_test_op.dart (+38 -0)
📝 frontend/appflowy_flutter/lib/plugins/database/application/field/type_option/type_option_data_parser.dart (+7 -0)
📝 frontend/appflowy_flutter/lib/plugins/database/application/row/row_service.dart (+5 -0)
📝 frontend/appflowy_flutter/lib/plugins/database/board/application/board_bloc.dart (+17 -4)
📝 frontend/appflowy_flutter/lib/plugins/database/board/presentation/board_page.dart (+50 -12)
📝 frontend/appflowy_flutter/lib/plugins/database/widgets/field/type_option_editor/date/date_time_format.dart (+30 -5)
📝 frontend/appflowy_flutter/lib/plugins/database/widgets/group/database_group.dart (+125 -10)
📝 frontend/appflowy_flutter/pubspec.lock (+8 -0)
📝 frontend/appflowy_flutter/pubspec.yaml (+2 -0)
📝 frontend/appflowy_flutter/test/bloc_test/board_test/create_card_test.dart (+106 -0)
📝 frontend/resources/translations/en.json (+3 -0)
📝 frontend/rust-lib/flowy-database2/src/entities/board_entities.rs (+10 -0)
📝 frontend/rust-lib/flowy-database2/src/services/database/database_editor.rs (+1 -1)
📝 frontend/rust-lib/flowy-database2/src/services/database_view/view_editor.rs (+22 -1)
📝 frontend/rust-lib/flowy-database2/src/services/setting/entities.rs (+12 -0)
📝 frontend/rust-lib/flowy-database2/tests/database/layout_test/test.rs (+1 -1)

📄 Description

On submitting a new row in board view, if the setting is enabled and the text is a URL fetch url metadata and fill the row with it.

closes #5963

Feature Preview

https://github.com/user-attachments/assets/26ffc1c1-e009-4079-be28-a1c92c3a1861

PR Checklist

My code adheres to AppFlowy's Conventions
I've listed at least one issue that this PR fixes in the description above.
I've added a test(s) to validate changes in this PR, or this PR only contains semantic changes.
All existing tests are passing.

_{🔄 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/6130 **Author:** [@zoli](https://github.com/zoli) **Created:** 8/30/2024 **Status:** 🔄 Open **Base:** `main` ← **Head:** `feat/fetch-url-meta-data` --- ### 📝 Commits (6) - [`7057439`](https://github.com/AppFlowy-IO/AppFlowy/commit/7057439684baafac143a187e54071c86672cec97) feat: fetch url meta data on creating row - [`ec9bcca`](https://github.com/AppFlowy-IO/AppFlowy/commit/ec9bcca2e588cc1a8c418aa94246ddcf82084af8) style: merge main - [`43f4305`](https://github.com/AppFlowy-IO/AppFlowy/commit/43f43054f27b277b20afc3392f8e7fbe631bb186) style: merge main - [`c1abc5d`](https://github.com/AppFlowy-IO/AppFlowy/commit/c1abc5db9f467e9dd4fc418b908d796370045e12) fix: url meta fetching state - [`5ed1724`](https://github.com/AppFlowy-IO/AppFlowy/commit/5ed17241b69b48899f8f0d6e0b9b6304057cae3d) feat: select url field to fill on fetching meta - [`347d0fc`](https://github.com/AppFlowy-IO/AppFlowy/commit/347d0fc6f6c207219013a95cf3561560838f591e) test: added integration and bloc test for fetching url meta data ### 📊 Changes **17 files changed** (+504 additions, -34 deletions) <details> <summary>View changed files</summary> 📝 `frontend/appflowy_flutter/integration_test/desktop/board/board_add_row_test.dart` (+67 -0) 📝 `frontend/appflowy_flutter/integration_test/shared/database_test_op.dart` (+38 -0) 📝 `frontend/appflowy_flutter/lib/plugins/database/application/field/type_option/type_option_data_parser.dart` (+7 -0) 📝 `frontend/appflowy_flutter/lib/plugins/database/application/row/row_service.dart` (+5 -0) 📝 `frontend/appflowy_flutter/lib/plugins/database/board/application/board_bloc.dart` (+17 -4) 📝 `frontend/appflowy_flutter/lib/plugins/database/board/presentation/board_page.dart` (+50 -12) 📝 `frontend/appflowy_flutter/lib/plugins/database/widgets/field/type_option_editor/date/date_time_format.dart` (+30 -5) 📝 `frontend/appflowy_flutter/lib/plugins/database/widgets/group/database_group.dart` (+125 -10) 📝 `frontend/appflowy_flutter/pubspec.lock` (+8 -0) 📝 `frontend/appflowy_flutter/pubspec.yaml` (+2 -0) 📝 `frontend/appflowy_flutter/test/bloc_test/board_test/create_card_test.dart` (+106 -0) 📝 `frontend/resources/translations/en.json` (+3 -0) 📝 `frontend/rust-lib/flowy-database2/src/entities/board_entities.rs` (+10 -0) 📝 `frontend/rust-lib/flowy-database2/src/services/database/database_editor.rs` (+1 -1) 📝 `frontend/rust-lib/flowy-database2/src/services/database_view/view_editor.rs` (+22 -1) 📝 `frontend/rust-lib/flowy-database2/src/services/setting/entities.rs` (+12 -0) 📝 `frontend/rust-lib/flowy-database2/tests/database/layout_test/test.rs` (+1 -1) </details> ### 📄 Description On submitting a new row in board view, if the setting is enabled and the text is a URL fetch url metadata and fill the row with it. closes #5963 ### Feature Preview https://github.com/user-attachments/assets/26ffc1c1-e009-4079-be28-a1c92c3a1861 #### PR Checklist - [x] My code adheres to [AppFlowy's Conventions](https://docs.appflowy.io/docs/documentation/software-contributions/conventions) - [x] I've listed at least one issue that this PR fixes in the description above. - [ ] I've added a test(s) to validate changes in this PR, or this PR only contains semantic changes. - [ ] All existing tests are passing. --- <sub>🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.</sub>