fix: mark already sync snapshot as clean after pushing

Author: GursheenK

frontend/src/stores/saving.js

@@ -103,32 +103,39 @@ let syncThumbnail = 0
 
 const syncOfflineStatus = ref(null)
 
-const syncPresentationToServer = async (hadDroppedConnection) => {
-	isSaving.value = true
+const syncSnapshotToServer = async (wasConnectionRestored, snapshot) => {
+	// if connection was restored, syncing was not triggered by auto-save
+	// so add badge after coming back online
+	if (wasConnectionRestored) {
+		syncOfflineStatus.value = 'Syncing local changes'
+	}
 
-	try {
-		const snapshot = await getPresentationFromLocalDB(presentationId.value)
+	await savePresentationDoc(snapshot.content)
 
-		if (!snapshot || !snapshot.dirty) return
+	if (wasConnectionRestored) {
+		syncOfflineStatus.value = 'Changes synced'
+		setTimeout(() => {
+			syncOfflineStatus.value = null
+		}, 2000)
+	}
 
-		if (hadDroppedConnection) {
-			syncOfflineStatus.value = 'Syncing local changes'
-		}
+	// after successful sync, make sure local copy is marked as clean so it's not synced again
+	await savePresentationToLocalDB({
+		...snapshot,
+		dirty: false,
+		updatedAt: Date.now(),
+	})
+}
 
-		await savePresentationDoc(snapshot.content)
+const syncPresentationToServer = async (wasConnectionRestored) => {
+	isSaving.value = true
 
-		if (hadDroppedConnection) {
-			syncOfflineStatus.value = 'Changes synced'
-			setTimeout(() => {
-				syncOfflineStatus.value = null
-			}, 2000)
-		}
+	try {
+		const snapshot = await getPresentationFromLocalDB(presentationId.value)
+		if (!snapshot || !snapshot.dirty) return
 
-		await savePresentationToLocalDB({
-			...snapshot,
-			dirty: false,
-			updatedAt: Date.now(),
-		})
+		// if there's an unsynced snapshot locally, sync it to server
+		syncSnapshotToServer(wasConnectionRestored, snapshot)
 	} catch (err) {
 		console.error('Sync to server failed: ', err)
 	} finally {