fix(notification): the title is only part of the subject

There never was a title in the notification object.
nobe4 · May 21, 2024 · 2d78698 · 2d78698
1 parent 177198d
commit 2d78698
Show file tree

Hide file tree

Showing 2 changed files with 3 additions and 4 deletions.
diff --git a/config.sample.yaml b/config.sample.yaml
@@ -12,8 +12,8 @@ groups:
     filters:
       - .author.login == "dependabot[bot]"
       - >
-        (.title | contains("something unimportant")) or
-        (.title | contains("something already done"))
+        (.subject.title | contains("something unimportant")) or
+        (.subject.title | contains("something already done"))
 
   - name: ignore ci failures for the current repo
     action: done

diff --git a/internal/notifications/notifications.go b/internal/notifications/notifications.go
@@ -12,7 +12,6 @@ type Notifications []Notification
 
 type Notification struct {
 	// Standard API fields
-	Title      string     `json:"title"`
 	Id         string     `json:"id"`
 	Unread     bool       `json:"unread"`
 	Reason     string     `json:"reason"`
@@ -73,7 +72,7 @@ func (n NotificationMap) Uniq() NotificationMap {
 }
 
 func (n Notification) ToString() string {
-	return fmt.Sprintf("%s %+v", n.Id, n)
+	return fmt.Sprintf("[%s] %s by %s: '%s' ", n.Id, n.Repository.FullName, n.Author.Login, n.Subject.Title)
 }
 
 func (n NotificationMap) ToString() string {