publishMultiple: stop if nothing to publish

oleeskild · Sep 27, 2023 · f2e8b38 · f2e8b38
1 parent 1ec5236
commit f2e8b38
Show file tree

Hide file tree

Showing 2 changed files with 37 additions and 41 deletions.
diff --git a/main.ts b/main.ts
@@ -191,6 +191,18 @@ export default class DigitalGarden extends Plugin {
 					const filesToDelete = publishStatus.deletedNotePaths;
 					const imagesToDelete = publishStatus.deletedImagePaths;
 
+					const totalItems =
+						filesToPublish.length +
+						filesToDelete.length +
+						imagesToDelete.length;
+
+					if (totalItems === 0) {
+						new Notice("Garden is already fully synced!");
+						statusBarItem.remove();
+
+						return;
+					}
+
 					const statusBar = new PublishStatusBar(
 						statusBarItem,
 						filesToPublish.length +

diff --git a/src/publisher/Publisher.ts b/src/publisher/Publisher.ts
@@ -82,27 +82,7 @@ export default class Publisher {
 	}
 
 	async delete(path: string): Promise<boolean> {
-		if (!this.settings.githubRepo) {
-			new Notice(
-				"Config error: You need to define a GitHub repo in the plugin settings",
-			);
-			throw {};
-		}
-
-		if (!this.settings.githubUserName) {
-			new Notice(
-				"Config error: You need to define a GitHub Username in the plugin settings",
-			);
-			throw {};
-		}
-
-		if (!this.settings.githubToken) {
-			new Notice(
-				"Config error: You need to define a GitHub Token in the plugin settings",
-			);
-			throw {};
-		}
-
+		this.validateSettings();
 		const octokit = new Octokit({ auth: this.settings.githubToken });
 
 		const payload = {
@@ -169,26 +149,7 @@ export default class Publisher {
 	}
 
 	async uploadToGithub(path: string, content: string) {
-		if (!this.settings.githubRepo) {
-			new Notice(
-				"Config error: You need to define a GitHub repo in the plugin settings",
-			);
-			throw {};
-		}
-
-		if (!this.settings.githubUserName) {
-			new Notice(
-				"Config error: You need to define a GitHub Username in the plugin settings",
-			);
-			throw {};
-		}
-
-		if (!this.settings.githubToken) {
-			new Notice(
-				"Config error: You need to define a GitHub Token in the plugin settings",
-			);
-			throw {};
-		}
+		this.validateSettings();
 
 		const octokit = new Octokit({ auth: this.settings.githubToken });
 
@@ -245,4 +206,27 @@ export default class Publisher {
 			await this.uploadImage(image.path, image.content);
 		}
 	}
+
+	validateSettings() {
+		if (!this.settings.githubRepo) {
+			new Notice(
+				"Config error: You need to define a GitHub repo in the plugin settings",
+			);
+			throw {};
+		}
+
+		if (!this.settings.githubUserName) {
+			new Notice(
+				"Config error: You need to define a GitHub Username in the plugin settings",
+			);
+			throw {};
+		}
+
+		if (!this.settings.githubToken) {
+			new Notice(
+				"Config error: You need to define a GitHub Token in the plugin settings",
+			);
+			throw {};
+		}
+	}
 }