fix: add missing Discussions data (#438)

Author: codebytere

src/js/utils/github-api.test.ts

@@ -19,14 +19,15 @@ describe('./utils/github-api.ts', () => {
   });
 
   it('should get the notification type icon', () => {
+    expect(getNotificationTypeIcon('CheckSuite')).toBe('sync');
+    expect(getNotificationTypeIcon('Commit')).toBe('git-commit');
+    expect(getNotificationTypeIcon('Discussion')).toBe('comment-discussion');
     expect(getNotificationTypeIcon('Issue')).toBe('issue-opened');
     expect(getNotificationTypeIcon('PullRequest')).toBe('git-pull-request');
-    expect(getNotificationTypeIcon('Commit')).toBe('git-commit');
     expect(getNotificationTypeIcon('Release')).toBe('tag');
     expect(getNotificationTypeIcon('RepositoryVulnerabilityAlert')).toBe(
       'alert'
     );
-    expect(getNotificationTypeIcon('CheckSuite')).toBe('sync');
     expect(getNotificationTypeIcon('Unknown' as SubjectType)).toBe('question');
   });
 });

src/js/utils/github-api.ts

@@ -53,18 +53,20 @@ export function formatReason(
 
 export function getNotificationTypeIcon(type: SubjectType): string {
   switch (type) {
+    case 'CheckSuite':
+      return 'sync';
+    case 'Commit':
+      return 'git-commit';
+    case 'Discussion':
+      return 'comment-discussion';
     case 'Issue':
       return 'issue-opened';
     case 'PullRequest':
       return 'git-pull-request';
-    case 'Commit':
-      return 'git-commit';
     case 'Release':
       return 'tag';
     case 'RepositoryVulnerabilityAlert':
       return 'alert';
-    case 'CheckSuite':
-      return 'sync';
     default:
       return 'question';
   }

src/types/github.ts

@@ -13,11 +13,12 @@ export type Reason =
   | 'ci_activity';
 
 export type SubjectType =
+  | 'CheckSuite'
+  | 'Commit'
+  | 'Discussion'
   | 'Issue'
   | 'PullRequest'
-  | 'Commit'
   | 'Release'
-  | 'CheckSuite'
   | 'RepositoryVulnerabilityAlert';
 
 export interface Notification {