Improve reaction types

signalapp · Oct 13, 2021 · 21c00f5 · 21c00f5
1 parent 35a54cd
commit 21c00f5
Show file tree

Hide file tree

Showing 4 changed files with 5 additions and 5 deletions.
diff --git a/ts/messageModifiers/Reactions.ts b/ts/messageModifiers/Reactions.ts
@@ -13,7 +13,7 @@ export class ReactionModel extends Model<ReactionAttributesType> {}
 
 let singleton: Reactions | undefined;
 
-export class Reactions extends Collection {
+export class Reactions extends Collection<ReactionModel> {
   static getSingleton(): Reactions {
     if (!singleton) {
       singleton = new Reactions();

diff --git a/ts/model-types.d.ts b/ts/model-types.d.ts
@@ -373,7 +373,7 @@ export type ReactionAttributesType = {
   remove?: boolean;
   targetAuthorUuid: string;
   targetTimestamp: number;
-  fromId?: string;
+  fromId: string;
   timestamp: number;
   fromSync?: boolean;
 };
diff --git a/ts/models/conversations.ts b/ts/models/conversations.ts
@@ -4870,7 +4870,7 @@ export class ConversationModel extends window.Backbone
 
   async notify(
     message: Readonly<MessageModel>,
-    reaction?: WhatIsThis
+    reaction?: Readonly<ReactionModel>
   ): Promise<void> {
     // As a performance optimization don't perform any work if notifications are
     // disabled.

diff --git a/ts/models/messages.ts b/ts/models/messages.ts
@@ -108,7 +108,7 @@ import {
   MessageReceiptType,
 } from '../messageModifiers/MessageReceipts';
 import { Deletes } from '../messageModifiers/Deletes';
-import { Reactions } from '../messageModifiers/Reactions';
+import { Reactions, ReactionModel } from '../messageModifiers/Reactions';
 import { ReadSyncs } from '../messageModifiers/ReadSyncs';
 import { ViewSyncs } from '../messageModifiers/ViewSyncs';
 import { ViewOnceOpenSyncs } from '../messageModifiers/ViewOnceOpenSyncs';
@@ -3167,7 +3167,7 @@ export class MessageModel extends window.Backbone.Model<MessageAttributesType> {
   }
 
   async handleReaction(
-    reaction: typeof window.WhatIsThis,
+    reaction: ReactionModel,
     shouldPersist = true
   ): Promise<ReactionAttributesType | undefined> {
     const { attributes } = this;