[PR #7262] [MERGED] chore: support pasting image links that have no file extension #7813

New issue

Closed

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

mirror commented

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

Owner

📋 Pull Request Information

Original PR: https://github.com/AppFlowy-IO/AppFlowy/pull/7262
Author: @richardshiue
Created: 1/21/2025
Status: ✅ Merged
Merged: 1/22/2025
Merged by: @appflowy

Base: main ← Head: chore/image-urls-with-no-extension

📝 Commits (6)

acbbaac test: speed up copypasta tests
6a62955 chore: strip away markdown syntax if message is image only
333a1b7 chore: paste image urls with no file extension
37b4fd7 test: add integration test
81cddfe test: group tests
9374803 chore: apply code suggestions to 3 files

📊 Changes

4 files changed (+374 additions, -332 deletions)

View changed files

📝 frontend/appflowy_flutter/integration_test/desktop/document/document_copy_and_paste_test.dart (+310 -300)
📝 frontend/appflowy_flutter/lib/plugins/ai_chat/presentation/message/ai_message_action_bar.dart (+12 -1)
📝 frontend/appflowy_flutter/lib/plugins/document/presentation/editor_plugins/copy_and_paste/custom_paste_command.dart (+49 -31)
📝 frontend/appflowy_flutter/lib/shared/patterns/common_patterns.dart (+3 -0)

📄 Description

Note to reviewers:

It will be easier to review this PR going commit by commit.

The git diff for the test file looks messed up, but the only change in the most recent commit is to move some ungrouped tests into the group.

Feature Preview

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/7262 **Author:** [@richardshiue](https://github.com/richardshiue) **Created:** 1/21/2025 **Status:** ✅ Merged **Merged:** 1/22/2025 **Merged by:** [@appflowy](https://github.com/appflowy) **Base:** `main` ← **Head:** `chore/image-urls-with-no-extension` --- ### 📝 Commits (6) - [`acbbaac`](https://github.com/AppFlowy-IO/AppFlowy/commit/acbbaaced87ff92e7c6a7581e5f371538f300142) test: speed up copypasta tests - [`6a62955`](https://github.com/AppFlowy-IO/AppFlowy/commit/6a62955d36b23db5be453bf88d26af9a39dd812b) chore: strip away markdown syntax if message is image only - [`333a1b7`](https://github.com/AppFlowy-IO/AppFlowy/commit/333a1b7e3d34955afe5f5e5539fecccf6db1362d) chore: paste image urls with no file extension - [`37b4fd7`](https://github.com/AppFlowy-IO/AppFlowy/commit/37b4fd70b176ba2756efc90dc3f815109b427916) test: add integration test - [`81cddfe`](https://github.com/AppFlowy-IO/AppFlowy/commit/81cddfe983680d4124f23be6f25f2c2abf9bc409) test: group tests - [`9374803`](https://github.com/AppFlowy-IO/AppFlowy/commit/9374803f5da7aa83d890073674909841910c15af) chore: apply code suggestions to 3 files ### 📊 Changes **4 files changed** (+374 additions, -332 deletions) <details> <summary>View changed files</summary> 📝 `frontend/appflowy_flutter/integration_test/desktop/document/document_copy_and_paste_test.dart` (+310 -300) 📝 `frontend/appflowy_flutter/lib/plugins/ai_chat/presentation/message/ai_message_action_bar.dart` (+12 -1) 📝 `frontend/appflowy_flutter/lib/plugins/document/presentation/editor_plugins/copy_and_paste/custom_paste_command.dart` (+49 -31) 📝 `frontend/appflowy_flutter/lib/shared/patterns/common_patterns.dart` (+3 -0) </details> ### 📄 Description Note to reviewers: It will be easier to review this PR going commit by commit. The git diff for the test file looks messed up, but the only change in the most recent commit is to move some ungrouped tests into the group. ### Feature Preview  ---  #### PR Checklist - [x] My code adheres to [AppFlowy's Conventions](https://docs.appflowy.io/docs/documentation/software-contributions/conventions) - [ ] I've listed at least one issue that this PR fixes in the description above. - [x] I've added a test(s) to validate changes in this PR, or this PR only contains semantic changes. - [x] All existing tests are passing. --- <sub>🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.</sub>