feat: control access level in share menu (#8017)

* feat: use auto_confirm in share with user api

* feat: reduce share api calls

* feat: add team space section in title bar

* feat: only user with full access can invite guest/member

* feat: customize popup style

* feat: support section type

* feat: update access level based on public or private space

* test: add access level tests

* chore: update i18n

* chore: disable bloc talker

* fix: tests

* fix: general access test

* fix: view title bar test

* chore: i18n

Author: LucasXu0

frontend/appflowy_flutter/lib/features/share_tab/data/models/share_access_level.dart

@@ -32,13 +32,13 @@ enum ShareAccessLevel {
   String get subtitle {
     switch (this) {
       case ShareAccessLevel.readOnly:
-        return 'Can\'t make changes';
+        return LocaleKeys.shareTab_cantMakeChanges.tr();
       case ShareAccessLevel.readAndComment:
-        return 'Can make any changes';
+        return LocaleKeys.shareTab_canMakeAnyChanges.tr();
       case ShareAccessLevel.readAndWrite:
-        return 'Can make any changes';
+        return LocaleKeys.shareTab_canMakeAnyChanges.tr();
       case ShareAccessLevel.fullAccess:
-        return 'Can make any changes';
+        return LocaleKeys.shareTab_canMakeAnyChanges.tr();
     }
   }
 

frontend/appflowy_flutter/lib/features/share_tab/data/models/share_section_type.dart

@@ -4,6 +4,7 @@
 /// - shared: the shared section is shared, anyone in the shared section can view/edit it.
 /// - private: the shared section is private, only the users in the shared section can view/edit it.
 enum SharedSectionType {
+  unknown,
   public,
   shared,
   private;

frontend/appflowy_flutter/lib/features/share_tab/data/repositories/local_share_with_user_repository_impl.dart

@@ -2,6 +2,7 @@ import 'dart:math';
 
 import 'package:appflowy/features/share_tab/data/models/models.dart';
 import 'package:appflowy_backend/protobuf/flowy-error/errors.pb.dart';
+import 'package:appflowy_backend/protobuf/flowy-user/user_profile.pb.dart';
 import 'package:appflowy_result/appflowy_result.dart';
 
 import 'share_with_user_repository.dart';
@@ -11,41 +12,47 @@ class LocalShareWithUserRepositoryImpl extends ShareWithUserRepository {
   LocalShareWithUserRepositoryImpl();
 
   final SharedUsers _sharedUsers = [
+    // current user has full access
     SharedUser(
       email: 'lucas.xu@appflowy.io',
       name: 'Lucas Xu - Long long long long long name',
-      accessLevel: ShareAccessLevel.fullAccess,
+      accessLevel: ShareAccessLevel.readOnly,
       role: ShareRole.guest,
       avatarUrl: 'https://avatar.iran.liara.run/public',
     ),
+    // member user has read and write access
     SharedUser(
       email: 'vivian@appflowy.io',
       name: 'Vivian Wang',
       accessLevel: ShareAccessLevel.readAndWrite,
-      role: ShareRole.guest,
+      role: ShareRole.member,
       avatarUrl: 'https://avatar.iran.liara.run/public/girl',
     ),
+    // member user has read access
     SharedUser(
       email: 'shuheng@appflowy.io',
       name: 'Shuheng',
-      accessLevel: ShareAccessLevel.fullAccess,
-      role: ShareRole.owner,
+      accessLevel: ShareAccessLevel.readOnly,
+      role: ShareRole.member,
       avatarUrl: 'https://avatar.iran.liara.run/public/boy',
     ),
+    // guest user has read access
     SharedUser(
       email: 'guest_user_1@appflowy.io',
-      name: 'Guest User 1 - Long long long long long name',
+      name: 'Read Only Guest',
       accessLevel: ShareAccessLevel.readOnly,
       role: ShareRole.guest,
       avatarUrl: 'https://avatar.iran.liara.run/public/boy/10',
     ),
+    // guest user has read and write access
     SharedUser(
       email: 'guest_user_2@appflowy.io',
-      name: 'Guest User 2',
-      accessLevel: ShareAccessLevel.readOnly,
-      role: ShareRole.owner,
+      name: 'Read And Write Guest',
+      accessLevel: ShareAccessLevel.readAndWrite,
+      role: ShareRole.guest,
       avatarUrl: 'https://avatar.iran.liara.run/public/boy/11',
     ),
+    // Others
     SharedUser(
       email: 'member_user_1@appflowy.io',
       name: 'Member User 1',
@@ -157,4 +164,21 @@ class LocalShareWithUserRepositoryImpl extends ShareWithUserRepository {
 
     return FlowySuccess(null);
   }
+
+  @override
+  Future<FlowyResult<UserProfilePB, FlowyError>> getCurrentUserProfile() async {
+    // Simulate fetching current user profile
+    return FlowySuccess(
+      UserProfilePB()
+        ..email = 'lucas.xu@appflowy.io'
+        ..name = 'Lucas Xu',
+    );
+  }
+
+  @override
+  Future<FlowyResult<SharedSectionType, FlowyError>> getCurrentPageSectionType({
+    required String pageId,
+  }) async {
+    return FlowySuccess(SharedSectionType.private);
+  }
 }

frontend/appflowy_flutter/lib/features/share_tab/data/repositories/rust_share_with_user_repository_impl.dart

@@ -1,11 +1,15 @@
 import 'package:appflowy/features/share_tab/data/models/models.dart';
 import 'package:appflowy/features/util/extensions.dart';
+import 'package:appflowy/workspace/application/sidebar/space/space_bloc.dart';
+import 'package:appflowy/workspace/application/view/view_ext.dart';
 import 'package:appflowy_backend/dispatch/dispatch.dart';
 import 'package:appflowy_backend/log.dart';
 import 'package:appflowy_backend/protobuf/flowy-error/errors.pb.dart';
 import 'package:appflowy_backend/protobuf/flowy-folder/protobuf.dart';
+import 'package:appflowy_backend/protobuf/flowy-user/user_profile.pb.dart';
 import 'package:appflowy_backend/protobuf/flowy-user/workspace.pb.dart';
 import 'package:appflowy_result/appflowy_result.dart';
+import 'package:collection/collection.dart';
 
 import 'share_with_user_repository.dart';
 
@@ -70,6 +74,7 @@ class RustShareWithUserRepositoryImpl extends ShareWithUserRepository {
       viewId: pageId,
       emails: emails,
       accessLevel: accessLevel.accessLevel,
+      autoConfirm: true, // TODO: remove this after the backend is ready
     );
     final result = await FolderEventSharePageWithUser(request).send();
 
@@ -82,7 +87,9 @@ class RustShareWithUserRepositoryImpl extends ShareWithUserRepository {
         return FlowySuccess(success);
       },
       (failure) {
-        Log.error('sharePageWithUser: $failure');
+        Log.error(
+          'share page($pageId) with users($emails) with access level($accessLevel): $failure',
+        );
 
         return FlowyFailure(failure);
       },
@@ -124,4 +131,34 @@ class RustShareWithUserRepositoryImpl extends ShareWithUserRepository {
       },
     );
   }
+
+  @override
+  Future<FlowyResult<UserProfilePB, FlowyError>> getCurrentUserProfile() async {
+    final result = await UserEventGetUserProfile().send();
+    return result;
+  }
+
+  @override
+  Future<FlowyResult<SharedSectionType, FlowyError>> getCurrentPageSectionType({
+    required String pageId,
+  }) async {
+    final request = ViewIdPB.create()..value = pageId;
+    final result = await FolderEventGetViewAncestors(request).send();
+    final ancestors = result.fold(
+      (s) => s.items,
+      (f) => <ViewPB>[],
+    );
+    final space = ancestors.firstWhereOrNull((e) => e.isSpace);
+
+    if (space == null) {
+      return FlowySuccess(SharedSectionType.unknown);
+    }
+
+    final sectionType = switch (space.spacePermission) {
+      SpacePermission.publicToAll => SharedSectionType.public,
+      SpacePermission.private => SharedSectionType.private,
+    };
+
+    return FlowySuccess(sectionType);
+  }
 }

frontend/appflowy_flutter/lib/features/share_tab/data/repositories/share_with_user_repository.dart

@@ -1,5 +1,6 @@
 import 'package:appflowy/features/share_tab/data/models/models.dart';
 import 'package:appflowy_backend/protobuf/flowy-error/errors.pb.dart';
+import 'package:appflowy_backend/protobuf/flowy-user/user_profile.pb.dart';
 import 'package:appflowy_result/appflowy_result.dart';
 
 /// Abstract repository for sharing with users.
@@ -38,4 +39,12 @@ abstract class ShareWithUserRepository {
     required String email,
     required ShareRole role,
   });
+
+  /// Get current user profile.
+  Future<FlowyResult<UserProfilePB, FlowyError>> getCurrentUserProfile();
+
+  /// Get current page is in public section or private section.
+  Future<FlowyResult<SharedSectionType, FlowyError>> getCurrentPageSectionType({
+    required String pageId,
+  });
 }

frontend/appflowy_flutter/lib/features/share_tab/logic/share_tab_bloc.dart

@@ -6,7 +6,6 @@ import 'package:appflowy/plugins/document/presentation/editor_plugins/copy_and_p
 import 'package:appflowy/plugins/shared/share/constants.dart';
 import 'package:appflowy/shared/feature_flags.dart';
 import 'package:appflowy/startup/startup.dart';
-import 'package:appflowy/user/application/user_service.dart';
 import 'package:appflowy_result/appflowy_result.dart';
 import 'package:bloc/bloc.dart';
 
@@ -50,24 +49,33 @@ class ShareTabBloc extends Bloc<ShareTabEvent, ShareTabState> {
       return;
     }
 
-    final result = await UserBackendService.getCurrentUserProfile();
+    final result = await repository.getCurrentUserProfile();
     final currentUser = result.fold(
       (user) => user,
       (error) => null,
     );
 
+    final sectionTypeResult = await repository.getCurrentPageSectionType(
+      pageId: pageId,
+    );
+    final sectionType = sectionTypeResult.fold(
+      (type) => type,
+      (error) => SharedSectionType.unknown,
+    );
+
     final shareLink = ShareConstants.buildShareUrl(
       workspaceId: workspaceId,
       viewId: pageId,
     );
 
-    final users = await _getLatestSharedUsersOrCurrentUsers();
+    final users = await _getSharedUsers();
 
     emit(
       state.copyWith(
         currentUser: currentUser,
         shareLink: shareLink,
         users: users,
+        sectionType: sectionType,
       ),
     );
   }
@@ -124,7 +132,7 @@ class ShareTabBloc extends Bloc<ShareTabEvent, ShareTabState> {
 
     await result.fold(
       (_) async {
-        final users = await _getLatestSharedUsersOrCurrentUsers();
+        final users = await _getSharedUsers();
 
         emit(
           state.copyWith(
@@ -161,7 +169,7 @@ class ShareTabBloc extends Bloc<ShareTabEvent, ShareTabState> {
 
     await result.fold(
       (_) async {
-        final users = await _getLatestSharedUsersOrCurrentUsers();
+        final users = await _getSharedUsers();
         emit(
           state.copyWith(
             removeResult: FlowySuccess(null),
@@ -196,7 +204,7 @@ class ShareTabBloc extends Bloc<ShareTabEvent, ShareTabState> {
 
     await result.fold(
       (_) async {
-        final users = await _getLatestSharedUsersOrCurrentUsers();
+        final users = await _getSharedUsers();
         emit(
           state.copyWith(
             updateAccessLevelResult: FlowySuccess(null),
@@ -296,7 +304,7 @@ class ShareTabBloc extends Bloc<ShareTabEvent, ShareTabState> {
 
     await result.fold(
       (_) async {
-        final users = await _getLatestSharedUsersOrCurrentUsers();
+        final users = await _getSharedUsers();
         emit(
           state.copyWith(
             turnIntoMemberResult: FlowySuccess(null),
@@ -315,7 +323,7 @@ class ShareTabBloc extends Bloc<ShareTabEvent, ShareTabState> {
     );
   }
 
-  Future<SharedUsers> _getLatestSharedUsersOrCurrentUsers() async {
+  Future<SharedUsers> _getSharedUsers() async {
     final shareResult = await repository.getSharedUsersInPage(
       pageId: pageId,
     );

frontend/appflowy_flutter/lib/features/share_tab/logic/share_tab_state.dart

@@ -15,6 +15,7 @@ class ShareTabState {
     this.shareLink = '',
     this.generalAccessRole,
     this.linkCopied = false,
+    this.sectionType = SharedSectionType.private,
     this.initialResult,
     this.shareResult,
     this.removeResult,
@@ -30,6 +31,7 @@ class ShareTabState {
   final String shareLink;
   final ShareAccessLevel? generalAccessRole;
   final bool linkCopied;
+  final SharedSectionType sectionType;
   final FlowyResult<void, FlowyError>? initialResult;
   final FlowyResult<void, FlowyError>? shareResult;
   final FlowyResult<void, FlowyError>? removeResult;
@@ -45,6 +47,7 @@ class ShareTabState {
     String? shareLink,
     ShareAccessLevel? generalAccessRole,
     bool? linkCopied,
+    SharedSectionType? sectionType,
     FlowyResult<void, FlowyError>? initialResult,
     FlowyResult<void, FlowyError>? shareResult,
     FlowyResult<void, FlowyError>? removeResult,
@@ -60,6 +63,7 @@ class ShareTabState {
       shareLink: shareLink ?? this.shareLink,
       generalAccessRole: generalAccessRole ?? this.generalAccessRole,
       linkCopied: linkCopied ?? this.linkCopied,
+      sectionType: sectionType ?? this.sectionType,
       initialResult: initialResult,
       shareResult: shareResult,
       removeResult: removeResult,
@@ -80,6 +84,7 @@ class ShareTabState {
         other.shareLink == shareLink &&
         other.generalAccessRole == generalAccessRole &&
         other.linkCopied == linkCopied &&
+        other.sectionType == sectionType &&
         other.initialResult == initialResult &&
         other.shareResult == shareResult &&
         other.removeResult == removeResult &&
@@ -98,6 +103,7 @@ class ShareTabState {
       shareLink,
       generalAccessRole,
       linkCopied,
+      sectionType,
       initialResult,
       shareResult,
       removeResult,
@@ -108,6 +114,6 @@ class ShareTabState {
 
   @override
   String toString() {
-    return 'ShareTabState(currentUser: $currentUser, users: $users, availableUsers: $availableUsers, isLoading: $isLoading, errorMessage: $errorMessage, shareLink: $shareLink, generalAccessRole: $generalAccessRole, linkCopied: $linkCopied, initialResult: $initialResult, shareResult: $shareResult, removeResult: $removeResult, updateAccessLevelResult: $updateAccessLevelResult, turnIntoMemberResult: $turnIntoMemberResult)';
+    return 'ShareTabState(currentUser: $currentUser, users: $users, availableUsers: $availableUsers, isLoading: $isLoading, errorMessage: $errorMessage, shareLink: $shareLink, generalAccessRole: $generalAccessRole, shareSectionType: $SharedSectionType, linkCopied: $linkCopied, initialResult: $initialResult, shareResult: $shareResult, removeResult: $removeResult, updateAccessLevelResult: $updateAccessLevelResult, turnIntoMemberResult: $turnIntoMemberResult)';
   }
 }