[PR #6548] [MERGED] feat: support 'share link' and 'publish' on mobile #7452

New issue

Closed

opened 2026-03-23 23:19:41 +00:00 by mirror · 0 comments

mirror commented

2026-03-23 23:19:41 +00:00

Owner

📋 Pull Request Information

Original PR: https://github.com/AppFlowy-IO/AppFlowy/pull/6548
Author: @LucasXu0
Created: 10/15/2024
Status: ✅ Merged
Merged: 10/15/2024
Merged by: @LucasXu0

Base: main ← Head: share_link_on_mobile

📝 Commits (10+)

3bb4ed6 feat: support publish on mobile
ddfd83a chore: replace icons
9a5564f Merge branch 'main' into publish_on_mobile
cd6e442 Merge branch 'main' into publish_on_mobile
af165a4 feat: support publish and copy link on mobile
f74dff8 feat: support copy link
63373fd feat: support copy share link
bd23381 chore: update icons
4099212 fix: flutter analyze
1dcec1f fix: ignore opening same page on mobile

📊 Changes

31 files changed (+369 additions, -48 deletions)

View changed files

📝 frontend/appflowy_flutter/integration_test/desktop/cloud/document/document_copy_link_to_block_test.dart (+2 -3)
➕ frontend/appflowy_flutter/integration_test/mobile/cloud/document/share_link_test.dart (+69 -0)
📝 frontend/appflowy_flutter/integration_test/shared/document_test_operations.dart (+21 -0)
📝 frontend/appflowy_flutter/lib/mobile/presentation/base/view_page/app_bar_buttons.dart (+6 -0)
📝 frontend/appflowy_flutter/lib/mobile/presentation/base/view_page/more_bottom_sheet.dart (+127 -15)
📝 frontend/appflowy_flutter/lib/mobile/presentation/bottom_sheet/bottom_sheet_view_page.dart (+68 -2)
📝 frontend/appflowy_flutter/lib/mobile/presentation/home/mobile_home_page.dart (+0 -1)
📝 frontend/appflowy_flutter/lib/mobile/presentation/setting/workspace/invite_members_screen.dart (+0 -1)
📝 frontend/appflowy_flutter/lib/mobile/presentation/widgets/flowy_mobile_quick_action_button.dart (+5 -2)
📝 frontend/appflowy_flutter/lib/plugins/document/presentation/editor_plugins/file/file_block_component.dart (+0 -1)
📝 frontend/appflowy_flutter/lib/plugins/document/presentation/editor_plugins/file/file_util.dart (+0 -1)
📝 frontend/appflowy_flutter/lib/plugins/document/presentation/editor_plugins/image/custom_image_block_component/image_menu.dart (+0 -1)
📝 frontend/appflowy_flutter/lib/plugins/document/presentation/editor_plugins/mention/mention_page_block.dart (+5 -2)
📝 frontend/appflowy_flutter/lib/plugins/document/presentation/editor_plugins/openai/widgets/smart_edit_node_widget.dart (+0 -1)
📝 frontend/appflowy_flutter/lib/shared/patterns/common_patterns.dart (+2 -2)
📝 frontend/appflowy_flutter/lib/user/presentation/helpers/handle_open_workspace_error.dart (+0 -1)
📝 frontend/appflowy_flutter/lib/user/presentation/screens/sign_in_screen/widgets/magic_link_sign_in_buttons.dart (+0 -1)
📝 frontend/appflowy_flutter/lib/util/share_log_files.dart (+0 -1)
📝 frontend/appflowy_flutter/lib/workspace/application/view/view_bloc.dart (+7 -3)
📝 frontend/appflowy_flutter/lib/workspace/presentation/home/menu/sidebar/workspace/sidebar_workspace.dart (+0 -1)

...and 11 more files

📄 Description

Feature Preview

Completed part of https://github.com/AppFlowy-IO/AppFlowy/issues/5997

support 'share link' on mobile
support 'publish' and 'visit site' on mobile
support 'unpublish' on mobile
tests

https://github.com/user-attachments/assets/1f7f8e53-c71c-479f-98f8-60bec10048f8

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/6548 **Author:** [@LucasXu0](https://github.com/LucasXu0) **Created:** 10/15/2024 **Status:** ✅ Merged **Merged:** 10/15/2024 **Merged by:** [@LucasXu0](https://github.com/LucasXu0) **Base:** `main` ← **Head:** `share_link_on_mobile` --- ### 📝 Commits (10+) - [`3bb4ed6`](https://github.com/AppFlowy-IO/AppFlowy/commit/3bb4ed6b7fb26e326153ec31b34d94e7befdcd78) feat: support publish on mobile - [`ddfd83a`](https://github.com/AppFlowy-IO/AppFlowy/commit/ddfd83afd1e2f9184780a281c458718e9f48af22) chore: replace icons - [`9a5564f`](https://github.com/AppFlowy-IO/AppFlowy/commit/9a5564f95cbe0eff422948bb3721185d7c050225) Merge branch 'main' into publish_on_mobile - [`cd6e442`](https://github.com/AppFlowy-IO/AppFlowy/commit/cd6e442c5e8cdefc00181259b5b6d1cf52f258f6) Merge branch 'main' into publish_on_mobile - [`af165a4`](https://github.com/AppFlowy-IO/AppFlowy/commit/af165a43afdc11e13174bde949c27944ec5e4974) feat: support publish and copy link on mobile - [`f74dff8`](https://github.com/AppFlowy-IO/AppFlowy/commit/f74dff8744a882f896a456725414a9bf6f69f5ef) feat: support copy link - [`63373fd`](https://github.com/AppFlowy-IO/AppFlowy/commit/63373fd130f1449f63b65f3c5e504ab1e89ca93b) feat: support copy share link - [`bd23381`](https://github.com/AppFlowy-IO/AppFlowy/commit/bd233817bf57934a1e85f2c024157fa1864e2702) chore: update icons - [`4099212`](https://github.com/AppFlowy-IO/AppFlowy/commit/40992124648484dce20392cead4def0ac42b3479) fix: flutter analyze - [`1dcec1f`](https://github.com/AppFlowy-IO/AppFlowy/commit/1dcec1f5a589f5a9e94bf1522377e6c78eee47f8) fix: ignore opening same page on mobile ### 📊 Changes **31 files changed** (+369 additions, -48 deletions) <details> <summary>View changed files</summary> 📝 `frontend/appflowy_flutter/integration_test/desktop/cloud/document/document_copy_link_to_block_test.dart` (+2 -3) ➕ `frontend/appflowy_flutter/integration_test/mobile/cloud/document/share_link_test.dart` (+69 -0) 📝 `frontend/appflowy_flutter/integration_test/shared/document_test_operations.dart` (+21 -0) 📝 `frontend/appflowy_flutter/lib/mobile/presentation/base/view_page/app_bar_buttons.dart` (+6 -0) 📝 `frontend/appflowy_flutter/lib/mobile/presentation/base/view_page/more_bottom_sheet.dart` (+127 -15) 📝 `frontend/appflowy_flutter/lib/mobile/presentation/bottom_sheet/bottom_sheet_view_page.dart` (+68 -2) 📝 `frontend/appflowy_flutter/lib/mobile/presentation/home/mobile_home_page.dart` (+0 -1) 📝 `frontend/appflowy_flutter/lib/mobile/presentation/setting/workspace/invite_members_screen.dart` (+0 -1) 📝 `frontend/appflowy_flutter/lib/mobile/presentation/widgets/flowy_mobile_quick_action_button.dart` (+5 -2) 📝 `frontend/appflowy_flutter/lib/plugins/document/presentation/editor_plugins/file/file_block_component.dart` (+0 -1) 📝 `frontend/appflowy_flutter/lib/plugins/document/presentation/editor_plugins/file/file_util.dart` (+0 -1) 📝 `frontend/appflowy_flutter/lib/plugins/document/presentation/editor_plugins/image/custom_image_block_component/image_menu.dart` (+0 -1) 📝 `frontend/appflowy_flutter/lib/plugins/document/presentation/editor_plugins/mention/mention_page_block.dart` (+5 -2) 📝 `frontend/appflowy_flutter/lib/plugins/document/presentation/editor_plugins/openai/widgets/smart_edit_node_widget.dart` (+0 -1) 📝 `frontend/appflowy_flutter/lib/shared/patterns/common_patterns.dart` (+2 -2) 📝 `frontend/appflowy_flutter/lib/user/presentation/helpers/handle_open_workspace_error.dart` (+0 -1) 📝 `frontend/appflowy_flutter/lib/user/presentation/screens/sign_in_screen/widgets/magic_link_sign_in_buttons.dart` (+0 -1) 📝 `frontend/appflowy_flutter/lib/util/share_log_files.dart` (+0 -1) 📝 `frontend/appflowy_flutter/lib/workspace/application/view/view_bloc.dart` (+7 -3) 📝 `frontend/appflowy_flutter/lib/workspace/presentation/home/menu/sidebar/workspace/sidebar_workspace.dart` (+0 -1) _...and 11 more files_ </details> ### 📄 Description   ### Feature Preview Completed part of https://github.com/AppFlowy-IO/AppFlowy/issues/5997 - [x] support 'share link' on mobile - [x] support 'publish' and 'visit site' on mobile - [x] support 'unpublish' on mobile - [x] tests https://github.com/user-attachments/assets/1f7f8e53-c71c-479f-98f8-60bec10048f8  ---  #### 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. - [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>