Make default notification setting for conversations 'mentions only'.

The current default of 'all' is very noisy, especially when restoring an
account or linking a new device, which causes all instantiated
conversations to start sending notifications.

Author: ianmacd

ts/components/leftpane/conversation-list-item/HeaderItem.tsx

@@ -44,15 +44,14 @@ const NotificationSettingIcon = () => {
         />
       );
     case 'mentions_only':
+    default:
       return (
         <SessionIcon
           iconType="bell"
           iconColor={'var(--conversation-tab-text-color)'}
           iconSize="small"
         />
       );
-    default:
-      return null;
   }
 };
 

ts/hooks/useParamSelector.ts

@@ -111,7 +111,7 @@ export function useHasNickname(convoId?: string) {
 
 export function useNotificationSetting(convoId?: string) {
   const convoProps = useConversationPropsById(convoId);
-  return convoProps?.currentNotificationSetting || 'all';
+  return convoProps?.currentNotificationSetting || 'mentions_only';
 }
 export function useIsPublic(convoId?: string) {
   const convoProps = useConversationPropsById(convoId);

ts/models/conversationAttributes.ts

@@ -124,7 +124,7 @@ export const fillConvoAttributesWithDefaults = (
     lastMessageStatus: undefined,
     lastMessage: null,
 
-    triggerNotificationsFor: 'all', // if the settings is not set in the db, this is the default
+    triggerNotificationsFor: 'mentions_only', // if the settings is not set in the db, this is the default
 
     isTrustedForAttachmentDownload: false, // we don't trust a contact until we say so
     isApproved: false,

ts/node/database_utility.ts

@@ -141,7 +141,7 @@ export function formatRowOfConversation(
   }
 
   if (!convo.triggerNotificationsFor) {
-    convo.triggerNotificationsFor = 'all';
+    convo.triggerNotificationsFor = 'mentions_only';
   }
 
   if (!convo.lastJoinedTimestamp) {

ts/node/migration/sessionMigrations.ts

@@ -755,7 +755,7 @@ function updateToSessionSchemaVersion24(currentVersion: number, db: BetterSqlite
          ALTER TABLE ${CONVERSATIONS_TABLE} ADD COLUMN avatarHash TEXT; -- only used for opengroup avatar.
          ALTER TABLE ${CONVERSATIONS_TABLE} ADD COLUMN nickname TEXT;
          ALTER TABLE ${CONVERSATIONS_TABLE} ADD COLUMN profileKey TEXT;
-         ALTER TABLE ${CONVERSATIONS_TABLE} ADD COLUMN triggerNotificationsFor TEXT DEFAULT "all";
+         ALTER TABLE ${CONVERSATIONS_TABLE} ADD COLUMN triggerNotificationsFor TEXT DEFAULT "mentions_only";
          ALTER TABLE ${CONVERSATIONS_TABLE} ADD COLUMN isTrustedForAttachmentDownload INTEGER DEFAULT "FALSE";
          ALTER TABLE ${CONVERSATIONS_TABLE} ADD COLUMN isPinned INTEGER DEFAULT "FALSE";
          ALTER TABLE ${CONVERSATIONS_TABLE} ADD COLUMN isApproved INTEGER DEFAULT "FALSE";

ts/test/session/unit/models/ConversationModels_test.ts

@@ -149,7 +149,7 @@ describe('fillConvoAttributesWithDefaults', () => {
     it('initialize triggerNotificationsFor if not given', () => {
       expect(fillConvoAttributesWithDefaults({} as ConversationAttributes)).to.have.deep.property(
         'triggerNotificationsFor',
-        'all'
+        'mentions_only'
       );
     });