Fix sending empty notifications (#19589)

- Don't send empty notifications on read notifications API.
3 years ago · c8ec2261a9
parent 438646e0b5
commit c8ec2261a9
1 changed files with 1 additions and 1 deletions
--- a/routers/api/v1/notify/repo.go
+++ b/routers/api/v1/notify/repo.go
@ -214,7 +214,7 @@ func ReadRepoNotifications(ctx *context.APIContext) {
 		targetStatus = models.NotificationStatusRead
 	}

-	changed := make([]*structs.NotificationThread, len(nl))
+	changed := make([]*structs.NotificationThread, 0, len(nl))

 	for _, n := range nl {
 		notif, err := models.SetNotificationStatus(n.ID, ctx.Doer, targetStatus)