Set shouldPush to false to reactions

xmtp · Dec 4, 2024 · 09a4af8 · 09a4af8
1 parent 5b3d404
commit 09a4af8
Show file tree

Hide file tree

Showing 2 changed files with 4 additions and 20 deletions.
diff --git a/content-types/content-type-reaction/src/Reaction.test.ts b/content-types/content-type-reaction/src/Reaction.test.ts
@@ -97,24 +97,8 @@ describe("ReactionContentType", () => {
     expect(messageContent.schema).toBe("shortcode");
   });
 
-  it("has a proper shouldPush value based on content", () => {
+  it("has a proper shouldPush value", () => {
     const codec = new ReactionCodec();
-
-    const addReaction: Reaction = {
-      action: "added",
-      content: "smile",
-      reference: "foo",
-      schema: "shortcode",
-    };
-
-    const removeReaction: Reaction = {
-      action: "removed",
-      content: "smile",
-      reference: "foo",
-      schema: "shortcode",
-    };
-
-    expect(codec.shouldPush(addReaction)).toBe(true);
-    expect(codec.shouldPush(removeReaction)).toBe(false);
+    expect(codec.shouldPush()).toBe(false);
   });
 });
diff --git a/content-types/content-type-reaction/src/Reaction.ts b/content-types/content-type-reaction/src/Reaction.ts
@@ -101,7 +101,7 @@ export class ReactionCodec
     }
   }
 
-  shouldPush(content: Reaction): boolean {
-    return content.action === "added";
+  shouldPush(): boolean {
+    return false;
   }
 }