Use thread ID and category for chat message pushes (#82)

Author: jeffyanta

pkg/code/push/data.go

@@ -115,6 +115,8 @@ func sendMutableNotificationToOwner(
 				ctx,
 				pushTokenRecord.PushToken,
 				titleKey,
+				string(notificationType),
+				titleKey, // All mutable pushes have a thread ID that's the title
 				kvs,
 			)
 		case push_data.TokenTypeFcmAndroid:

pkg/push/fcm/provider.go

@@ -89,7 +89,12 @@ func (p *provider) SendPush(ctx context.Context, pushToken, title, body string)
 }
 
 // SendMutableAPNSPush implements push.Provider.SendMutableAPNSPush
-func (p *provider) SendMutableAPNSPush(ctx context.Context, pushToken, titleKey string, kvs map[string]string) error {
+func (p *provider) SendMutableAPNSPush(
+	ctx context.Context,
+	pushToken,
+	titleKey, category, threadId string,
+	kvs map[string]string,
+) error {
 	defer metrics.TraceMethodCall(ctx, metricsStructName, "SendMutableAPNSPush").End()
 
 	_, err := p.client.Send(ctx, &messaging.Message{
@@ -102,6 +107,8 @@ func (p *provider) SendMutableAPNSPush(ctx context.Context, pushToken, titleKey
 						TitleLocKey: titleKey,
 						Body:        "...",
 					},
+					Category:       category,
+					ThreadID:       threadId,
 					MutableContent: true,
 				},
 			},

pkg/push/memory/provider.go

@@ -35,7 +35,12 @@ func (p *provider) SendPush(ctx context.Context, pushToken, title, body string)
 }
 
 // SendMutableAPNSPush implements push.Provider.SendMutableAPNSPush
-func (p *provider) SendMutableAPNSPush(ctx context.Context, pushToken, titleKey string, kvs map[string]string) error {
+func (p *provider) SendMutableAPNSPush(
+	ctx context.Context,
+	pushToken,
+	titleKey, category, threadId string,
+	kvs map[string]string,
+) error {
 	return simulateSendingPush(pushToken)
 }
 

pkg/push/provider.go

@@ -13,7 +13,12 @@ type Provider interface {
 
 	// SendMutableAPNSPush sends a push over APNS with a text body that's mutable
 	// on the client using custom key value pairs
-	SendMutableAPNSPush(ctx context.Context, pushToken, titleKey string, kvs map[string]string) error
+	SendMutableAPNSPush(
+		ctx context.Context,
+		pushToken,
+		titleKey, category, threadId string,
+		kvs map[string]string,
+	) error
 
 	// SendDataPush sends a data push
 	SendDataPush(ctx context.Context, pushToken string, kvs map[string]string) error