[PR #7788] [MERGED] refactor: local auth type #8113

New issue

Closed

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

mirror commented

2026-03-23 23:22:40 +00:00

Owner

📋 Pull Request Information

Original PR: https://github.com/AppFlowy-IO/AppFlowy/pull/7788
Author: @appflowy
Created: 4/20/2025
Status: ✅ Merged
Merged: 4/20/2025
Merged by: @appflowy

Base: main ← Head: local_auth_type

📝 Commits (7)

2f5b494 chore: remove authticator pb
2c5f41b fix: set auth type
ccd1f5f chore: revert pod lock
bb5d364 chore: fix test
833a2bf Merge branch 'main' into local_auth_type
747a63d chore: create user workspace for anon user
fd581b4 chore: clippy

📊 Changes

64 files changed (+328 additions, -329 deletions)

View changed files

📝 frontend/appflowy_flutter/ios/Podfile.lock (+23 -23)
📝 frontend/appflowy_flutter/lib/mobile/presentation/bottom_sheet/bottom_sheet_view_page.dart (+1 -2)
📝 frontend/appflowy_flutter/lib/mobile/presentation/home/mobile_home_page_header.dart (+1 -1)
📝 frontend/appflowy_flutter/lib/mobile/presentation/home/setting/settings_popup_menu.dart (+1 -1)
📝 frontend/appflowy_flutter/lib/mobile/presentation/home/tab/mobile_space_tab.dart (+1 -1)
📝 frontend/appflowy_flutter/lib/mobile/presentation/home/workspaces/workspace_menu_bottom_sheet.dart (+1 -1)
📝 frontend/appflowy_flutter/lib/mobile/presentation/setting/user_session_setting_group.dart (+1 -1)
📝 frontend/appflowy_flutter/lib/plugins/ai_chat/chat_page.dart (+1 -1)
📝 frontend/appflowy_flutter/lib/plugins/database/application/sync/database_sync_bloc.dart (+1 -1)
📝 frontend/appflowy_flutter/lib/plugins/database/widgets/row/row_banner.dart (+2 -3)
📝 frontend/appflowy_flutter/lib/plugins/document/application/document_bloc.dart (+2 -2)
📝 frontend/appflowy_flutter/lib/plugins/document/application/document_collaborators_bloc.dart (+1 -2)
📝 frontend/appflowy_flutter/lib/plugins/document/application/document_sync_bloc.dart (+1 -2)
📝 frontend/appflowy_flutter/lib/plugins/document/presentation/editor_plugins/file/file_util.dart (+3 -3)
📝 frontend/appflowy_flutter/lib/plugins/document/presentation/editor_plugins/page_style/_page_style_cover_image.dart (+1 -2)
📝 frontend/appflowy_flutter/lib/plugins/shared/share/share_bloc.dart (+1 -1)
📝 frontend/appflowy_flutter/lib/shared/icon_emoji_picker/icon_uploader.dart (+3 -3)
📝 frontend/appflowy_flutter/lib/startup/deps_resolver.dart (+1 -1)
📝 frontend/appflowy_flutter/lib/startup/tasks/appflowy_cloud_task.dart (+1 -1)
📝 frontend/appflowy_flutter/lib/user/application/auth/af_cloud_auth_service.dart (+1 -1)

...and 44 more files

📄 Description

remove duplicated authenticator type
set auth type

Summary by Sourcery

Refactor and consolidate authentication type handling across the AppFlowy codebase, replacing the duplicated AuthenticatorPB with a more consistent AuthTypePB

Enhancements:

Standardize authentication type representation across different components
Simplify authentication type conversion logic
Remove redundant authentication type enum

Chores:

Update multiple files to use the new AuthTypePB enum
Rename authentication type enum values for clarity

_{🔄 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/7788 **Author:** [@appflowy](https://github.com/appflowy) **Created:** 4/20/2025 **Status:** ✅ Merged **Merged:** 4/20/2025 **Merged by:** [@appflowy](https://github.com/appflowy) **Base:** `main` ← **Head:** `local_auth_type` --- ### 📝 Commits (7) - [`2f5b494`](https://github.com/AppFlowy-IO/AppFlowy/commit/2f5b494885904aac89cd173d0eb0978c7a0bd9b7) chore: remove authticator pb - [`2c5f41b`](https://github.com/AppFlowy-IO/AppFlowy/commit/2c5f41b58062771135fa4dd77c6b3e8777c7c2db) fix: set auth type - [`ccd1f5f`](https://github.com/AppFlowy-IO/AppFlowy/commit/ccd1f5f8e9742a86c023664edb2d17e848a1c6ef) chore: revert pod lock - [`bb5d364`](https://github.com/AppFlowy-IO/AppFlowy/commit/bb5d36402aa1396d4b76429a7ddd4ec265cc5257) chore: fix test - [`833a2bf`](https://github.com/AppFlowy-IO/AppFlowy/commit/833a2bf5d6f00f9ab7dcbe4aab9163bd5139e522) Merge branch 'main' into local_auth_type - [`747a63d`](https://github.com/AppFlowy-IO/AppFlowy/commit/747a63d452507c201d2d0007a8fd2868f8cce9fc) chore: create user workspace for anon user - [`fd581b4`](https://github.com/AppFlowy-IO/AppFlowy/commit/fd581b44535894540c5e9e2a2f6240cb47cf0642) chore: clippy ### 📊 Changes **64 files changed** (+328 additions, -329 deletions) <details> <summary>View changed files</summary> 📝 `frontend/appflowy_flutter/ios/Podfile.lock` (+23 -23) 📝 `frontend/appflowy_flutter/lib/mobile/presentation/bottom_sheet/bottom_sheet_view_page.dart` (+1 -2) 📝 `frontend/appflowy_flutter/lib/mobile/presentation/home/mobile_home_page_header.dart` (+1 -1) 📝 `frontend/appflowy_flutter/lib/mobile/presentation/home/setting/settings_popup_menu.dart` (+1 -1) 📝 `frontend/appflowy_flutter/lib/mobile/presentation/home/tab/mobile_space_tab.dart` (+1 -1) 📝 `frontend/appflowy_flutter/lib/mobile/presentation/home/workspaces/workspace_menu_bottom_sheet.dart` (+1 -1) 📝 `frontend/appflowy_flutter/lib/mobile/presentation/setting/user_session_setting_group.dart` (+1 -1) 📝 `frontend/appflowy_flutter/lib/plugins/ai_chat/chat_page.dart` (+1 -1) 📝 `frontend/appflowy_flutter/lib/plugins/database/application/sync/database_sync_bloc.dart` (+1 -1) 📝 `frontend/appflowy_flutter/lib/plugins/database/widgets/row/row_banner.dart` (+2 -3) 📝 `frontend/appflowy_flutter/lib/plugins/document/application/document_bloc.dart` (+2 -2) 📝 `frontend/appflowy_flutter/lib/plugins/document/application/document_collaborators_bloc.dart` (+1 -2) 📝 `frontend/appflowy_flutter/lib/plugins/document/application/document_sync_bloc.dart` (+1 -2) 📝 `frontend/appflowy_flutter/lib/plugins/document/presentation/editor_plugins/file/file_util.dart` (+3 -3) 📝 `frontend/appflowy_flutter/lib/plugins/document/presentation/editor_plugins/page_style/_page_style_cover_image.dart` (+1 -2) 📝 `frontend/appflowy_flutter/lib/plugins/shared/share/share_bloc.dart` (+1 -1) 📝 `frontend/appflowy_flutter/lib/shared/icon_emoji_picker/icon_uploader.dart` (+3 -3) 📝 `frontend/appflowy_flutter/lib/startup/deps_resolver.dart` (+1 -1) 📝 `frontend/appflowy_flutter/lib/startup/tasks/appflowy_cloud_task.dart` (+1 -1) 📝 `frontend/appflowy_flutter/lib/user/application/auth/af_cloud_auth_service.dart` (+1 -1) _...and 44 more files_ </details> ### 📄 Description 1. remove duplicated authenticator type 2. set auth type ## Summary by Sourcery Refactor and consolidate authentication type handling across the AppFlowy codebase, replacing the duplicated `AuthenticatorPB` with a more consistent `AuthTypePB` Enhancements: - Standardize authentication type representation across different components - Simplify authentication type conversion logic - Remove redundant authentication type enum Chores: - Update multiple files to use the new `AuthTypePB` enum - Rename authentication type enum values for clarity --- <sub>🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.</sub>