Don't fetch posts text as it's not used

Joystream · Mar 3, 2024 · 2cd3f72 · 2cd3f72
1 parent b1bdc8f
commit 2cd3f72
Show file tree

Hide file tree

Showing 3 changed files with 1 addition and 16 deletions.
diff --git a/packages/server/src/notifier/model/email/utils/forum.ts b/packages/server/src/notifier/model/email/utils/forum.ts
@@ -7,7 +7,6 @@ interface ForumPost {
   author: string
   threadId: string
   thread: string
-  text: string
 }
 const cachedForumPosts: { [id: string]: ForumPost } = {}
 export const getForumPost = async (id: string): Promise<ForumPost> => {
@@ -21,7 +20,6 @@ export const getForumPost = async (id: string): Promise<ForumPost> => {
       author: post.author.handle,
       threadId: post.thread.id,
       thread: post.thread.title,
-      text: post.text,
     }
   }
 
@@ -31,7 +29,6 @@ export const getForumPost = async (id: string): Promise<ForumPost> => {
 interface ForumThread {
   author: string
   title: string
-  text?: string
 }
 const cachedForumThreads: { [id: string]: ForumThread } = {}
 export const getForumThread = async (id: string): Promise<ForumThread> => {
@@ -44,7 +41,6 @@ export const getForumThread = async (id: string): Promise<ForumThread> => {
     cachedForumThreads[id] = {
       author: thread.author.handle,
       title: thread.title,
-      text: thread.initialPost?.text,
     }
   }
 

diff --git a/packages/server/src/notifier/queries/entities/forum.graphql b/packages/server/src/notifier/queries/entities/forum.graphql
@@ -7,7 +7,6 @@ query GetPost($id: ID!) {
       id
       title
     }
-    text
   }
 }
 
@@ -17,9 +16,6 @@ query GetThread($id: ID!) {
       handle
     }
     title
-    initialPost {
-      text
-    }
   }
 }
 

diff --git a/packages/server/test/notifier.test.ts b/packages/server/test/notifier.test.ts
@@ -102,7 +102,6 @@ describe('Notifier', () => {
             forumPostByUniqueInput: {
               author: { handle: 'author:handle' },
               thread: { id: 'thread:id', title: 'thread:title' },
-              text: 'Lorem Ipsum',
             },
           })
 
@@ -306,7 +305,6 @@ describe('Notifier', () => {
             forumPostByUniqueInput: {
               author: { handle: 'author:handle' },
               thread: { id: 'thread:id', title: 'thread:title' },
-              text: 'Lorem Ipsum',
             },
           })
         // -------------------
@@ -385,11 +383,7 @@ describe('Notifier', () => {
                 return { forumCategoryByUniqueInput: { parentId: variables.id === 'bar' ? 'foo' : null } }
               case GetThreadDocument:
                 return {
-                  forumThreadByUniqueInput: {
-                    author: { handle: 'author:handle' },
-                    title: `${variables.id}:title`,
-                    initialPost: { text: 'Lorem Ipsum' },
-                  },
+                  forumThreadByUniqueInput: { author: { handle: 'author:handle' }, title: `${variables.id}:title` },
                 }
             }
           })
@@ -812,7 +806,6 @@ describe('Notifier', () => {
           forumPostByUniqueInput: {
             author: { handle: 'author:handle' },
             thread: { id: 'thread:id', title: 'thread:title' },
-            text: 'Lorem Ipsum',
           },
         })
-Original file line number
+Diff line change
@@ Expand Up / @@ -7,7 +7,6 @@ query GetPost($id: ID!) { @@
           id
           title
         }
-        text
       }
     }
@@ Expand All / @@ -17,9 +16,6 @@ query GetThread($id: ID!) { @@
           handle
         }
         title
-        initialPost {
-          text
-        }
       }
     }
@@ Expand Down @@