Further events adding and fixes (#2372)

Author: mdmohsin7

app/lib/pages/apps/widgets/app_section_card.dart

@@ -7,6 +7,7 @@ import 'package:omi/pages/apps/app_detail/app_detail.dart';
 import 'package:omi/providers/app_provider.dart';
 import 'package:omi/utils/analytics/mixpanel.dart';
 import 'package:omi/utils/other/temp.dart';
+import 'package:omi/widgets/extensions/string.dart';
 import 'package:provider/provider.dart';
 
 class AppSectionCard extends StatelessWidget {
@@ -146,7 +147,7 @@ class SectionAppItemCard extends StatelessWidget {
                     Padding(
                       padding: const EdgeInsets.only(top: 2.0),
                       child: Text(
-                        app.getCategoryName(),
+                        app.description.decodeString,
                         maxLines: 1,
                         overflow: TextOverflow.ellipsis,
                         style: const TextStyle(color: Colors.grey, fontSize: 13),

app/lib/pages/apps/widgets/create_options_sheet.dart

@@ -1,5 +1,6 @@
 import 'package:flutter/material.dart';
 import 'package:omi/pages/apps/add_app.dart';
+import 'package:omi/pages/persona/persona_profile.dart';
 import 'package:omi/pages/persona/persona_provider.dart';
 import 'package:omi/providers/home_provider.dart';
 import 'package:provider/provider.dart';
@@ -64,8 +65,16 @@ class CreateOptionsSheet extends StatelessWidget {
                 MixpanelManager().pageOpened('Create Persona');
                 // Set routing in provider and navigate to Persona Profile page
                 Provider.of<PersonaProvider>(context, listen: false).setRouting(PersonaProfileRouting.create_my_clone);
-                Provider.of<HomeProvider>(context, listen: false).setIndex(3);
-                Provider.of<HomeProvider>(context, listen: false).onSelectedIndexChanged!(3);
+                Navigator.of(context).push(
+                  MaterialPageRoute(
+                    builder: (context) => const PersonaProfilePage(),
+                    settings: const RouteSettings(
+                      arguments: 'from_settings',
+                    ),
+                  ),
+                );
+                // Provider.of<HomeProvider>(context, listen: false).setIndex(3);
+                // Provider.of<HomeProvider>(context, listen: false).onSelectedIndexChanged!(3);
               },
             ),
           ),

app/lib/pages/memories/memories_review_page.dart

@@ -63,7 +63,7 @@ class _MemoriesReviewPageState extends State<MemoriesReviewPage> {
     // Process memories with a small delay to allow UI to update
     for (var memory in memoriesToProcess) {
       await Future.delayed(const Duration(milliseconds: 20));
-      context.read<MemoriesProvider>().reviewMemory(memory, approve);
+      context.read<MemoriesProvider>().reviewMemory(memory, approve, 'review_page_batch');
     }
 
     setState(() {
@@ -104,7 +104,7 @@ class _MemoriesReviewPageState extends State<MemoriesReviewPage> {
     setState(() => _isProcessing = true);
 
     // Process the single memory
-    context.read<MemoriesProvider>().reviewMemory(memory, approve);
+    context.read<MemoriesProvider>().reviewMemory(memory, approve, 'review_page_single');
 
     setState(() {
       remainingMemories.remove(memory);

app/lib/pages/memories/page.dart

@@ -145,6 +145,7 @@ class MemoriesPageState extends State<MemoriesPage> with AutomaticKeepAliveClien
                                                   onPressed: () {
                                                     _searchController.clear();
                                                     provider.setSearchQuery('');
+                                                    MixpanelManager().memorySearchCleared(provider.memories.length);
                                                   },
                                                 )
                                               ]
@@ -161,6 +162,11 @@ class MemoriesPageState extends State<MemoriesPage> with AutomaticKeepAliveClien
                                           ),
                                         ),
                                         onChanged: (value) => provider.setSearchQuery(value),
+                                        onSubmitted: (value) {
+                                          if (value.isNotEmpty) {
+                                            MixpanelManager().memorySearched(value, provider.filteredMemories.length);
+                                          }
+                                        },
                                       ),
                                     ),
                                   );
@@ -303,7 +309,10 @@ class MemoriesPageState extends State<MemoriesPage> with AutomaticKeepAliveClien
                               return MemoryItem(
                                 memory: memory,
                                 provider: provider,
-                                onTap: _showQuickEditSheet,
+                                onTap: (BuildContext context, Memory tappedMemory, MemoriesProvider tappedProvider) {
+                                  MixpanelManager().memoryListItemClicked(tappedMemory);
+                                  _showQuickEditSheet(context, tappedMemory, tappedProvider);
+                                },
                               );
                             },
                           ),
@@ -400,6 +409,7 @@ class MemoriesPageState extends State<MemoriesPage> with AutomaticKeepAliveClien
   }
 
   void _showMemoryManagementSheet(BuildContext context, MemoriesProvider provider) {
+    MixpanelManager().memoriesManagementSheetOpened();
     showModalBottomSheet(
       context: context,
       backgroundColor: Colors.transparent,

app/lib/pages/memories/widgets/memory_review_sheet.dart

@@ -29,6 +29,7 @@ class MemoriesReviewSheet extends StatelessWidget {
             margin: const EdgeInsets.only(bottom: 12),
             child: Column(
               children: [
+                const SizedBox(height: 10),
                 Row(
                   mainAxisAlignment: MainAxisAlignment.spaceBetween,
                   children: [

app/lib/providers/memories_provider.dart

@@ -3,6 +3,7 @@ import 'package:omi/backend/http/api/memories.dart';
 import 'package:omi/backend/preferences.dart';
 import 'package:omi/backend/schema/memory.dart';
 import 'package:omi/providers/base_provider.dart';
+import 'package:omi/utils/analytics/mixpanel.dart';
 import 'package:tuple/tuple.dart';
 import 'package:uuid/uuid.dart';
 import 'package:flutter/foundation.dart';
@@ -86,9 +87,13 @@ class MemoriesProvider extends ChangeNotifier {
   }
 
   void deleteAllMemories() async {
+    final int countBeforeDeletion = _memories.length;
     await deleteAllMemoriesServer();
     _memories.clear();
     _unreviewed.clear();
+    if (countBeforeDeletion > 0) {
+      MixpanelManager().memoriesAllDeleted(countBeforeDeletion);
+    }
     _setCategories();
   }
 
@@ -117,9 +122,12 @@ class MemoriesProvider extends ChangeNotifier {
 
     final idx = _memories.indexWhere((m) => m.id == memory.id);
     if (idx != -1) {
-      memory.visibility = visibility;
-      _memories[idx] = memory;
-      _unreviewed.remove(memory);
+      Memory memoryToUpdate = _memories[idx];
+      memoryToUpdate.visibility = visibility;
+      _memories[idx] = memoryToUpdate;
+      _unreviewed.removeWhere((m) => m.id == memory.id);
+
+      MixpanelManager().memoryVisibilityChanged(memoryToUpdate, visibility);
       _setCategories();
     }
   }
@@ -147,7 +155,9 @@ class MemoriesProvider extends ChangeNotifier {
     }
   }
 
-  void reviewMemory(Memory memory, bool approved) async {
+  void reviewMemory(Memory memory, bool approved, String source) async {
+    MixpanelManager().memoryReviewed(memory, approved, source);
+
     await reviewMemoryServer(memory.id, approved);
 
     final idx = _memories.indexWhere((m) => m.id == memory.id);
@@ -176,13 +186,29 @@ class MemoriesProvider extends ChangeNotifier {
 
   Future<void> updateAllMemoriesVisibility(bool makePrivate) async {
     final visibility = makePrivate ? MemoryVisibility.private : MemoryVisibility.public;
+    int updatedCount = 0;
+    List<Memory> memoriesSuccessfullyUpdated = [];
 
-    for (var memory in _memories) {
+    for (var memory in List.from(_memories)) {
       if (memory.visibility != visibility) {
-        await updateMemoryVisibility(memory, visibility);
+        try {
+          await updateMemoryVisibilityServer(memory.id, visibility.name);
+          final idx = _memories.indexWhere((m) => m.id == memory.id);
+          if (idx != -1) {
+            _memories[idx].visibility = visibility;
+            memoriesSuccessfullyUpdated.add(_memories[idx]);
+            updatedCount++;
+          }
+        } catch (e) {
+          print('Failed to update visibility for memory ${memory.id}: $e');
+        }
       }
     }
 
-    notifyListeners();
+    if (updatedCount > 0) {
+      MixpanelManager().memoriesAllVisibilityChanged(visibility, updatedCount);
+    }
+
+    _setCategories();
   }
 }

app/lib/utils/analytics/mixpanel.dart

@@ -167,6 +167,56 @@ class MixpanelManager {
   void memoriesPageCreatedMemory(MemoryCategory category) =>
       track('Fact Page Created Fact', properties: {'fact_category': category.toString().split('.').last});
 
+  void memorySearched(String query, int resultsCount) {
+    track('Fact Searched', properties: {
+      'search_query_length': query.length,
+      'results_count': resultsCount,
+    });
+  }
+
+  void memorySearchCleared(int totalFactsCount) {
+    track('Fact Search Cleared', properties: {'total_facts_count': totalFactsCount});
+  }
+
+  void memoryListItemClicked(Memory memory) {
+    track('Fact List Item Clicked', properties: {
+      'fact_id': memory.id,
+      'fact_category': memory.category.toString().split('.').last,
+    });
+  }
+
+  void memoryVisibilityChanged(Memory memory, MemoryVisibility newVisibility) {
+    track('Fact Visibility Changed', properties: {
+      'fact_id': memory.id,
+      'fact_category': memory.category.toString().split('.').last,
+      'new_visibility': newVisibility.name,
+    });
+  }
+
+  void memoriesAllVisibilityChanged(MemoryVisibility newVisibility, int count) {
+    track('All Facts Visibility Changed', properties: {
+      'new_visibility': newVisibility.name,
+      'facts_count': count,
+    });
+  }
+
+  void memoryReviewed(Memory memory, bool approved, String source) {
+    track('Fact Reviewed', properties: {
+      'fact_id': memory.id,
+      'fact_category': memory.category.toString().split('.').last,
+      'status': approved ? 'approved' : 'discarded',
+      'source': source,
+    });
+  }
+
+  void memoriesAllDeleted(int countBeforeDeletion) {
+    track('All Facts Deleted', properties: {
+      'facts_count_before_deletion': countBeforeDeletion,
+    });
+  }
+
+  void memoriesManagementSheetOpened() => track('Facts Management Sheet Opened');
+
   Map<String, dynamic> _getTranscriptProperties(String transcript) {
     String transcriptCopy = transcript.substring(0, transcript.length);
     int speakersCount = 0;