imterah/element-portable
1
0
Fork 0
Code Issues Pull requests Projects Releases Packages Wiki Activity Actions

use correct default for notification silencing (#9388)

Browse source
This commit is contained in:
Kerry 2022-10-11 18:11:11 +02:00 • committed by GitHub
parent 7c1c49540a
commit 9e055ee99d
No known key found for this signature in database
GPG key ID: 4AEE18F83AFDEB23
2 changed files with 3 additions and 3 deletions
Download patch file Download diff file Expand all files Collapse all files

2
src/utils/notifications.ts
View file

@ -51,5 +51,5 @@ export async function createLocalNotificationSettingsIfNeeded(cli: MatrixClient)
export function localNotificationsAreSilenced(cli: MatrixClient): boolean { export function localNotificationsAreSilenced(cli: MatrixClient): boolean {
const eventType = getLocalNotificationAccountDataEventType(cli.deviceId); const eventType = getLocalNotificationAccountDataEventType(cli.deviceId);
const event = cli.getAccountData(eventType); const event = cli.getAccountData(eventType);
return event?.getContent<LocalNotificationSettings>()?.is_silenced ?? true; return event?.getContent<LocalNotificationSettings>()?.is_silenced ?? false;
} }

4
test/utils/notifications-test.ts
View file

@ -81,8 +81,8 @@ describe('notifications', () => {
}); });
describe('localNotificationsAreSilenced', () => { describe('localNotificationsAreSilenced', () => {
it('defaults to true when no setting exists', () => { it('defaults to false when no setting exists', () => {
expect(localNotificationsAreSilenced(mockClient)).toBeTruthy(); expect(localNotificationsAreSilenced(mockClient)).toBeFalsy();
}); });
it('checks the persisted value', () => { it('checks the persisted value', () => {
mockClient.setAccountData(accountDataEventKey, { is_silenced: true }); mockClient.setAccountData(accountDataEventKey, { is_silenced: true });