[PR #405] [MERGED] Put business logic in services2 #4119

Closed
opened 2026-03-23 21:36:23 +00:00 by mirror · 0 comments
Owner

📋 Pull Request Information

Original PR: https://github.com/AppFlowy-IO/AppFlowy/pull/405
Author: @MikeWallaceDev
Created: 3/1/2022
Status: Merged
Merged: 3/2/2022
Merged by: @appflowy

Base: mainHead: put_business_logic_in_services2


📝 Commits (5)

  • 33c758e refactor: put Auth business logic in a service
  • 853d9e8 refactor: put UserSettings business logic in a service
  • c2f2ad2 refactor: Empty out infrastructure folder
  • 31467d3 refactor: move resolvers to /startup
  • be14a6b refactor: renamed helper.dart to notification_helper.dart

📊 Changes

23 files changed (+41 additions, -41 deletions)

View changed files

📝 frontend/app_flowy/lib/core/network_monitor.dart (+0 -0)
📝 frontend/app_flowy/lib/core/notification_helper.dart (+0 -0)
📝 frontend/app_flowy/lib/startup/home_deps_resolver.dart (+0 -0)
📝 frontend/app_flowy/lib/startup/startup.dart (+2 -2)
📝 frontend/app_flowy/lib/startup/tasks/app_widget.dart (+2 -2)
📝 frontend/app_flowy/lib/startup/tasks/platform_service.dart (+1 -1)
📝 frontend/app_flowy/lib/startup/user_deps_resolver.dart (+6 -6)
📝 frontend/app_flowy/lib/user/application/auth_service.dart (+1 -1)
📝 frontend/app_flowy/lib/user/application/sign_in_bloc.dart (+4 -4)
📝 frontend/app_flowy/lib/user/application/sign_up_bloc.dart (+4 -4)
📝 frontend/app_flowy/lib/user/application/user_listener.dart (+1 -1)
📝 frontend/app_flowy/lib/user/application/user_settings_service.dart (+1 -1)
📝 frontend/app_flowy/lib/user/presentation/router.dart (+2 -2)
📝 frontend/app_flowy/lib/user/presentation/sign_in_screen.dart (+1 -1)
📝 frontend/app_flowy/lib/user/presentation/sign_up_screen.dart (+1 -1)
📝 frontend/app_flowy/lib/user/presentation/skip_log_in_screen.dart (+5 -5)
📝 frontend/app_flowy/lib/user/presentation/splash_screen.dart (+1 -1)
📝 frontend/app_flowy/lib/workspace/application/app/app_listener.dart (+1 -1)
📝 frontend/app_flowy/lib/workspace/application/appearance.dart (+2 -2)
📝 frontend/app_flowy/lib/workspace/application/trash/trash_listener.dart (+1 -1)

...and 3 more files

📄 Description

Same as other one, each commit is a repo -> service


🔄 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/405 **Author:** [@MikeWallaceDev](https://github.com/MikeWallaceDev) **Created:** 3/1/2022 **Status:** ✅ Merged **Merged:** 3/2/2022 **Merged by:** [@appflowy](https://github.com/appflowy) **Base:** `main` ← **Head:** `put_business_logic_in_services2` --- ### 📝 Commits (5) - [`33c758e`](https://github.com/AppFlowy-IO/AppFlowy/commit/33c758e71190c25ec12857b1a2712cce0cfdaaf0) refactor: put Auth business logic in a service - [`853d9e8`](https://github.com/AppFlowy-IO/AppFlowy/commit/853d9e896863355527bc8f1c03b37e74540bd94d) refactor: put UserSettings business logic in a service - [`c2f2ad2`](https://github.com/AppFlowy-IO/AppFlowy/commit/c2f2ad24dba3dd74a0bc9de1a3ae8cd2a37f17bf) refactor: Empty out infrastructure folder - [`31467d3`](https://github.com/AppFlowy-IO/AppFlowy/commit/31467d3e98a59ea39c967ce8b0a52fbeeb397d6e) refactor: move resolvers to /startup - [`be14a6b`](https://github.com/AppFlowy-IO/AppFlowy/commit/be14a6b0ad70d5eed6dbfb3c162c041272977dc7) refactor: renamed helper.dart to notification_helper.dart ### 📊 Changes **23 files changed** (+41 additions, -41 deletions) <details> <summary>View changed files</summary> 📝 `frontend/app_flowy/lib/core/network_monitor.dart` (+0 -0) 📝 `frontend/app_flowy/lib/core/notification_helper.dart` (+0 -0) 📝 `frontend/app_flowy/lib/startup/home_deps_resolver.dart` (+0 -0) 📝 `frontend/app_flowy/lib/startup/startup.dart` (+2 -2) 📝 `frontend/app_flowy/lib/startup/tasks/app_widget.dart` (+2 -2) 📝 `frontend/app_flowy/lib/startup/tasks/platform_service.dart` (+1 -1) 📝 `frontend/app_flowy/lib/startup/user_deps_resolver.dart` (+6 -6) 📝 `frontend/app_flowy/lib/user/application/auth_service.dart` (+1 -1) 📝 `frontend/app_flowy/lib/user/application/sign_in_bloc.dart` (+4 -4) 📝 `frontend/app_flowy/lib/user/application/sign_up_bloc.dart` (+4 -4) 📝 `frontend/app_flowy/lib/user/application/user_listener.dart` (+1 -1) 📝 `frontend/app_flowy/lib/user/application/user_settings_service.dart` (+1 -1) 📝 `frontend/app_flowy/lib/user/presentation/router.dart` (+2 -2) 📝 `frontend/app_flowy/lib/user/presentation/sign_in_screen.dart` (+1 -1) 📝 `frontend/app_flowy/lib/user/presentation/sign_up_screen.dart` (+1 -1) 📝 `frontend/app_flowy/lib/user/presentation/skip_log_in_screen.dart` (+5 -5) 📝 `frontend/app_flowy/lib/user/presentation/splash_screen.dart` (+1 -1) 📝 `frontend/app_flowy/lib/workspace/application/app/app_listener.dart` (+1 -1) 📝 `frontend/app_flowy/lib/workspace/application/appearance.dart` (+2 -2) 📝 `frontend/app_flowy/lib/workspace/application/trash/trash_listener.dart` (+1 -1) _...and 3 more files_ </details> ### 📄 Description Same as other one, each commit is a repo -> service --- <sub>🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.</sub>
mirror 2026-03-23 21:36:23 +00:00
Sign in to join this conversation.
No milestone
No project
No assignees
1 participant
Notifications
Due date
The due date is invalid or out of range. Please use the format "yyyy-mm-dd".

No due date set.

Dependencies

No dependencies set.

Reference
AppFlowy-IO/AppFlowy#4119
No description provided.