Longer reports + bug where convo screen isn't popped upon report (#163)

duolicious · Dec 31, 2023 · c59101c · c59101c
1 parent dbb0fd7
commit c59101c
Show file tree

Hide file tree

Showing 3 changed files with 9 additions and 12 deletions.
diff --git a/components/conversation-screen.tsx b/components/conversation-screen.tsx
@@ -81,8 +81,7 @@ const Menu = ({navigation, name, personId, messages, closeFn}) => {
       name,
       personId,
       context: (
-        `Conversation Screen - ` +
-        `${String(JSON.stringify(messages, null, 2)).slice(0, 900)}`
+        `Conversation Screen\n${String(JSON.stringify(messages, null, 2))}`
       ),
     };
 
@@ -98,12 +97,6 @@ const Menu = ({navigation, name, personId, messages, closeFn}) => {
     })();
   }, [personId]);
 
-  useEffect(() => {
-    return listen(`skip-profile-${personId}`, () => {
-      navigation.popToTop();
-    });
-  }, [navigation, personId]);
-
   const pressableStyle: ViewStyle = {
     flexDirection: 'row',
     gap: 10,
@@ -375,6 +368,12 @@ const ConversationScreen = ({navigation, route}) => {
     setShowMenu(x => !x);
   }, []);
 
+  useEffect(() => {
+    return listen(`skip-profile-${personId}`, () => {
+      navigation.popToTop();
+    });
+  }, [navigation, personId]);
+
   return (
     <>
       <TopNavBar>

diff --git a/components/inbox-item.tsx b/components/inbox-item.tsx
@@ -103,9 +103,7 @@ const IntrosItem = ({
     const data: ReportModalInitialData = {
       name,
       personId,
-      context: (
-        `Inbox Item - ${JSON.stringify(contextData).slice(0, 900)}`
-      ),
+      context: `Inbox Item\n${JSON.stringify(contextData)}`,
     };
 
     notify('open-report-modal', data);

diff --git a/components/report-modal.tsx b/components/report-modal.tsx
@@ -56,7 +56,7 @@ const ReportModal = () => {
 
 
     setIsLoading(true);
-    const completeReportText = `${context} - ${reportText}`;
+    const completeReportText = `${context.slice(0, 7999)}\n${reportText}`;
     if (await setSkipped(personId, true, completeReportText)) {
       setIsVisible(false);
     } else {