Skip to content
This repository was archived by the owner on Sep 11, 2024. It is now read-only.

Commit ecc5e8b

Browse files
committed
Add logging in case of failure
1 parent 76f2a23 commit ecc5e8b

File tree

1 file changed

+14
-1
lines changed

1 file changed

+14
-1
lines changed

src/stores/ReleaseAnnouncementStore.ts

+14-1
Original file line numberDiff line numberDiff line change
@@ -17,6 +17,7 @@
1717
*/
1818

1919
import { TypedEventEmitter } from "matrix-js-sdk/src/matrix";
20+
import { logger } from "matrix-js-sdk/src/logger";
2021

2122
import SettingsStore from "../settings/SettingsStore";
2223
import { SettingLevel } from "../settings/SettingLevel";
@@ -93,9 +94,12 @@ export class ReleaseAnnouncementStore extends TypedEventEmitter<ReleaseAnnouncem
9394
*/
9495
public getReleaseAnnouncement(): Features | null {
9596
const viewedReleaseAnnouncements = this.getViewedReleaseAnnouncements();
97+
console.log("viewedReleaseAnnouncements", viewedReleaseAnnouncements);
9698

9799
// Find the first feature that has not been viewed
98100
for (let i = this.index; i < FEATURES.length; i++) {
101+
console.log("FEATURES[i]", FEATURES[i]);
102+
console.log("viewedReleaseAnnouncements[FEATURES[i]]", viewedReleaseAnnouncements[FEATURES[i]]);
99103
if (!viewedReleaseAnnouncements[FEATURES[i]]) {
100104
this.index = i;
101105
return FEATURES[this.index];
@@ -128,7 +132,16 @@ export class ReleaseAnnouncementStore extends TypedEventEmitter<ReleaseAnnouncem
128132

129133
const canSetValue = SettingsStore.canSetValue("releaseAnnouncement", null, SettingLevel.ACCOUNT);
130134
if (canSetValue) {
131-
await SettingsStore.setValue("releaseAnnouncement", null, SettingLevel.ACCOUNT, viewedReleaseAnnouncements);
135+
try {
136+
await SettingsStore.setValue(
137+
"releaseAnnouncement",
138+
null,
139+
SettingLevel.ACCOUNT,
140+
viewedReleaseAnnouncements,
141+
);
142+
} catch (e) {
143+
logger.log("Failed to set release announcement settings", e);
144+
}
132145
}
133146
}
134147

0 commit comments

Comments
 (0)