Merge remote-tracking branch 'origin/dev' into PE-4582-audio-preview-…

…in-app

android/fastlane/metadata/android/en-US/changelogs/62.txt

@@ -0,0 +1,3 @@
+- Adds an explanation for the privacy of drives on the Create Drive Modal
+- Starts tagging public pins with new GQL Tags: ArFS-Pin and Pinned-Data-Tx
+- Updates the icon for the Detach Drive option

android/fastlane/metadata/android/en-US/changelogs/63.txt

@@ -0,0 +1,4 @@
+- Integrates new translations.
+- Corrects buttons being overflowed on the modals.
+- Makes the Pricacy field of the “Create New Drive” modal not change color when focused.
+- Replaces the duplicate “More Info” button with Detach Drive on Mobile.

lib/blocs/drive_create/drive_create_cubit.dart

@@ -1,5 +1,9 @@
 import 'package:ardrive/blocs/blocs.dart';
-import 'package:ardrive/entities/entities.dart';
+import 'package:ardrive/core/arfs/entities/arfs_entities.dart'
+    show DrivePrivacy;
+import 'package:ardrive/entities/constants.dart' as constants;
+import 'package:ardrive/entities/drive_entity.dart';
+import 'package:ardrive/entities/folder_entity.dart';
 import 'package:ardrive/models/models.dart';
 import 'package:ardrive/services/services.dart';
 import 'package:ardrive/turbo/services/upload_service.dart';
@@ -15,7 +19,9 @@ part 'drive_create_state.dart';
 class DriveCreateCubit extends Cubit<DriveCreateState> {
   final form = FormGroup({
     'privacy': FormControl<String>(
-        value: DrivePrivacy.private, validators: [Validators.required]),
+      value: DrivePrivacy.private.name,
+      validators: [Validators.required],
+    ),
   });
 
   final ArweaveService _arweave;
@@ -35,25 +41,32 @@ class DriveCreateCubit extends Cubit<DriveCreateState> {
         _driveDao = driveDao,
         _profileCubit = profileCubit,
         _drivesCubit = drivesCubit,
-        super(DriveCreateInitial());
+        super(const DriveCreateInitial(privacy: DrivePrivacy.private));
+
+  void onPrivacyChanged() {
+    final privacy = form.control('privacy').value == DrivePrivacy.private.name
+        ? DrivePrivacy.private
+        : DrivePrivacy.public;
+    emit(state.copyWith(privacy: privacy));
+  }
 
   Future<void> submit(
     String driveName,
   ) async {
     final profile = _profileCubit.state as ProfileLoggedIn;
     if (await _profileCubit.logoutIfWalletMismatch()) {
-      emit(DriveCreateWalletMismatch());
+      emit(DriveCreateWalletMismatch(privacy: state.privacy));
       return;
     }
 
     final minimumWalletBalance = BigInt.from(10000000);
     if (profile.walletBalance <= minimumWalletBalance &&
         !_turboUploadService.useTurboUpload) {
-      emit(DriveCreateZeroBalance());
+      emit(DriveCreateZeroBalance(privacy: state.privacy));
       return;
     }
 
-    emit(DriveCreateInProgress());
+    emit(DriveCreateInProgress(privacy: state.privacy));
 
     try {
       final String drivePrivacy = form.control('privacy').value;
@@ -72,8 +85,8 @@ class DriveCreateCubit extends Cubit<DriveCreateState> {
         name: driveName,
         rootFolderId: createRes.rootFolderId,
         privacy: drivePrivacy,
-        authMode: drivePrivacy == DrivePrivacy.private
-            ? DriveAuthMode.password
+        authMode: drivePrivacy == constants.DrivePrivacy.private
+            ? constants.DriveAuthMode.password
             : null,
       );
 
@@ -140,12 +153,12 @@ class DriveCreateCubit extends Cubit<DriveCreateState> {
       addError(err);
     }
 
-    emit(DriveCreateSuccess());
+    emit(DriveCreateSuccess(privacy: state.privacy));
   }
 
   @override
   void onError(Object error, StackTrace stackTrace) {
-    emit(DriveCreateFailure());
+    emit(DriveCreateFailure(privacy: state.privacy));
     super.onError(error, stackTrace);
 
     logger.e('Failed to create drive', error, stackTrace);

lib/blocs/drive_create/drive_create_state.dart

@@ -2,18 +2,68 @@ part of 'drive_create_cubit.dart';
 
 @immutable
 abstract class DriveCreateState extends Equatable {
+  final DrivePrivacy privacy;
+
+  const DriveCreateState({required this.privacy});
+
+  DriveCreateState copyWith({DrivePrivacy? privacy}) {
+    throw UnimplementedError();
+  }
+
   @override
-  List<Object> get props => [];
+  List<Object> get props => [privacy];
 }
 
-class DriveCreateInitial extends DriveCreateState {}
+class DriveCreateInitial extends DriveCreateState {
+  const DriveCreateInitial({required super.privacy});
 
-class DriveCreateZeroBalance extends DriveCreateState {}
+  @override
+  DriveCreateInitial copyWith({DrivePrivacy? privacy}) {
+    return DriveCreateInitial(privacy: privacy ?? this.privacy);
+  }
+}
 
-class DriveCreateInProgress extends DriveCreateState {}
+class DriveCreateZeroBalance extends DriveCreateState {
+  const DriveCreateZeroBalance({required super.privacy});
 
-class DriveCreateSuccess extends DriveCreateState {}
+  @override
+  DriveCreateZeroBalance copyWith({DrivePrivacy? privacy}) {
+    return DriveCreateZeroBalance(privacy: privacy ?? this.privacy);
+  }
+}
+
+class DriveCreateInProgress extends DriveCreateState {
+  const DriveCreateInProgress({required super.privacy});
+
+  @override
+  DriveCreateInProgress copyWith({DrivePrivacy? privacy}) {
+    return DriveCreateInProgress(privacy: privacy ?? this.privacy);
+  }
+}
 
-class DriveCreateFailure extends DriveCreateState {}
+class DriveCreateSuccess extends DriveCreateState {
+  const DriveCreateSuccess({required super.privacy});
 
-class DriveCreateWalletMismatch extends DriveCreateState {}
+  @override
+  DriveCreateSuccess copyWith({DrivePrivacy? privacy}) {
+    return DriveCreateSuccess(privacy: privacy ?? this.privacy);
+  }
+}
+
+class DriveCreateFailure extends DriveCreateState {
+  const DriveCreateFailure({required super.privacy});
+
+  @override
+  DriveCreateFailure copyWith({DrivePrivacy? privacy}) {
+    return DriveCreateFailure(privacy: privacy ?? this.privacy);
+  }
+}
+
+class DriveCreateWalletMismatch extends DriveCreateState {
+  const DriveCreateWalletMismatch({required super.privacy});
+
+  @override
+  DriveCreateWalletMismatch copyWith({DrivePrivacy? privacy}) {
+    return DriveCreateWalletMismatch(privacy: privacy ?? this.privacy);
+  }
+}

lib/blocs/pin_file/pin_file_bloc.dart

@@ -3,7 +3,7 @@ import 'dart:async';
 import 'package:ardrive/blocs/blocs.dart';
 import 'package:ardrive/core/arfs/entities/arfs_entities.dart';
 import 'package:ardrive/core/crypto/crypto.dart';
-import 'package:ardrive/entities/file_entity.dart';
+import 'package:ardrive/entities/entities.dart' show EntityTag, FileEntity;
 import 'package:ardrive/entities/string_types.dart';
 import 'package:ardrive/misc/misc.dart';
 import 'package:ardrive/models/models.dart';
@@ -279,6 +279,7 @@ class PinFileBloc extends Bloc<PinFileEvent, PinFileState> {
   ) async {
     final stateAsPinFileFieldsValid = state as PinFileFieldsValid;
     final profileState = _profileCubit.state as ProfileLoggedIn;
+    final wallet = profileState.wallet;
 
     emit(PinFileCreating(
       id: stateAsPinFileFieldsValid.id,
@@ -311,13 +312,29 @@ class PinFileBloc extends Bloc<PinFileEvent, PinFileState> {
           .folderById(driveId: _driveId, folderId: _parentFolderId)
           .getSingle();
 
+      final isAPublicPin = fileKey == null;
+
       if (_turboUploadService.useTurboUpload) {
         final fileDataItem = await _arweave.prepareEntityDataItem(
           newFileEntity,
-          profileState.wallet,
+          wallet,
           key: fileKey,
+          skipSignature: true,
         );
 
+        if (isAPublicPin) {
+          fileDataItem.addTag(
+            EntityTag.arFsPin,
+            'true',
+          );
+          fileDataItem.addTag(
+            EntityTag.pinnedDataTx,
+            newFileEntity.dataTxId!,
+          );
+        }
+
+        await fileDataItem.sign(wallet);
+
         await _turboUploadService.postDataItem(
           dataItem: fileDataItem,
           wallet: profileState.wallet,
@@ -326,10 +343,24 @@ class PinFileBloc extends Bloc<PinFileEvent, PinFileState> {
       } else {
         final fileDataItem = await _arweave.prepareEntityTx(
           newFileEntity,
-          profileState.wallet,
+          wallet,
           fileKey,
+          skipSignature: true,
         );
 
+        if (isAPublicPin) {
+          fileDataItem.addTag(
+            EntityTag.arFsPin,
+            'true',
+          );
+          fileDataItem.addTag(
+            EntityTag.pinnedDataTx,
+            newFileEntity.dataTxId!,
+          );
+        }
+
+        await fileDataItem.sign(wallet);
+
         await _arweave.postTx(fileDataItem);
         newFileEntity.txId = fileDataItem.id;
       }

lib/components/drive_create_form.dart

@@ -1,4 +1,5 @@
 import 'package:ardrive/blocs/blocs.dart';
+import 'package:ardrive/core/arfs/entities/arfs_entities.dart';
 import 'package:ardrive/l11n/l11n.dart';
 import 'package:ardrive/models/models.dart';
 import 'package:ardrive/pages/congestion_warning_wrapper.dart';
@@ -73,6 +74,8 @@ class _DriveCreateFormState extends State<DriveCreateForm> {
               ],
             );
           } else {
+            final privacy = state.privacy;
+
             return ArDriveStandardModal(
               title: appLocalizationsOf(context).createDriveEmphasized,
               content: SizedBox(
@@ -108,24 +111,57 @@ class _DriveCreateFormState extends State<DriveCreateForm> {
                       ReactiveDropdownField(
                         formControlName: 'privacy',
                         decoration: InputDecoration(
-                            labelText: appLocalizationsOf(context).privacy),
+                          label: Text(
+                            appLocalizationsOf(context).privacy,
+                            style: ArDriveTheme.of(context)
+                                .themeData
+                                .textFieldTheme
+                                .inputTextStyle
+                                .copyWith(
+                                  color: ArDriveTheme.of(context)
+                                      .themeData
+                                      .colors
+                                      .themeFgDisabled,
+                                  fontSize: 16,
+                                ),
+                          ),
+                          focusedBorder: InputBorder.none,
+                        ),
                         showErrors: (control) =>
                             control.dirty && control.invalid,
                         validationMessages:
                             kValidationMessages(appLocalizationsOf(context)),
                         items: [
                           DropdownMenuItem(
-                            value: 'public',
+                            value: DrivePrivacy.public.name,
                             child: Text(appLocalizationsOf(context).public),
                           ),
                           DropdownMenuItem(
-                            value: 'private',
+                            value: DrivePrivacy.private.name,
                             child: Text(
                               appLocalizationsOf(context).private,
                             ),
                           )
                         ],
+                        onChanged: (_) {
+                          context.read<DriveCreateCubit>().onPrivacyChanged();
+                        },
                       ),
+                      const SizedBox(height: 32),
+                      Row(children: [
+                        if (privacy == DrivePrivacy.private)
+                          Flexible(
+                              child: Text(
+                            appLocalizationsOf(context)
+                                .drivePrivacyDescriptionPrivate,
+                          ))
+                        else
+                          Flexible(
+                              child: Text(
+                            appLocalizationsOf(context)
+                                .drivePrivacyDescriptionPublic,
+                          ))
+                      ]),
                     ],
                   ),
                 ),

lib/entities/constants.dart

@@ -30,6 +30,9 @@ class EntityTag {
   static const blockEnd = 'Block-End';
   static const dataStart = 'Data-Start';
   static const dataEnd = 'Data-End';
+
+  static const pinnedDataTx = 'Pinned-Data-Tx';
+  static const arFsPin = 'ArFS-Pin';
 }
 
 class ContentType {

lib/l10n/app_en.arb

@@ -523,6 +523,14 @@
   "@driveName": {
     "description": "The name of certain drive"
   },
+  "drivePrivacyDescriptionPrivate": "Private Drives offer state-of-the-art security, you control access.",
+  "@drivePrivacyDescriptionPrivate": {
+    "description": "Explains private drives."
+  },
+  "drivePrivacyDescriptionPublic": "Public Drives are discoverable, others can find and view the contents.",
+  "@drivePrivacyDescriptionPublic": {
+    "description": "Explains public drives."
+  },
   "driveRoot": "Drive root",
   "@driveRoot": {
     "description": "The folder entity that is at the top of the folder hierarchy"