-
Notifications
You must be signed in to change notification settings - Fork 39k
userDataSyncService.resetLocal doesn't appear to do anything #90555
Copy link
Copy link
Closed
Labels
bugIssue identified by VS Code Team member as probable bugIssue identified by VS Code Team member as probable bugsettings-syncverifiedVerification succeededVerification succeeded
Milestone
Description
The userDataSyncService.ts -> resetLocal function looks like it's missing a call to synchroniser.resetLocal(), see below.
async resetLocal(): Promise<void> {
await this.checkEnablement();
for (const synchroniser of this.synchronisers) {
try {
synchroniser.resetLocal(); <-- Suggest adding this here?
} catch (e) {
this.logService.error(`${synchroniser.source}: ${toErrorMessage(e)}`);
this.logService.error(e);
}
}
}
Reactions are currently unavailable
Metadata
Metadata
Assignees
Labels
bugIssue identified by VS Code Team member as probable bugIssue identified by VS Code Team member as probable bugsettings-syncverifiedVerification succeededVerification succeeded