Defer interactions before database writes (#76)

* Defer button interactions before doing database writes

* Discord.js, or one of its dependencies, yelled at me for running Node below 16.9. It's probs fine, but updating docs anyway

* v1.6.3

* Update test runner to Node 16.9

.github/workflows/test.yml

@@ -12,7 +12,7 @@ jobs:
 
     strategy:
       matrix:
-        node-version: [16.6.x]
+        node-version: [16.9.x]
         # See supported Node.js release schedule at https://nodejs.org/en/about/releases/
 
     steps:
@@ -33,10 +33,10 @@ jobs:
 
     steps:
       - uses: actions/checkout@v2
-      - name: Use Node.js 16.6.x (earliest version supported)
+      - name: Use Node.js 16.9.x (earliest version supported)
         uses: actions/setup-node@v1
         with:
-          node-version: 16.6.x
+          node-version: 16.9.x
       - run: npm ci
       - run: npm run build
       - run: npm run test:e2e

README.md

@@ -10,14 +10,14 @@ This project started out as a simple Discord bot to play around with APIs and th
 
 ## Prerequisites
 
-This project requires [NodeJS](https://nodejs.org/) (version 16.6 or later), [NPM](https://npmjs.org/), and a [Discord bot account token](https://www.howtogeek.com/364225/how-to-make-your-own-discord-bot/).
+This project requires [NodeJS](https://nodejs.org/) (version 16.9 or later), [NPM](https://npmjs.org/), and a [Discord bot account token](https://www.howtogeek.com/364225/how-to-make-your-own-discord-bot/).
 To make sure you have them available on your machine,
 try running the following command:
 
 ```sh
 $ npm -v && node -v
 7.20.3
-v16.6.2
+v16.15.1
 ```
 
 ## Clone the Repo

package.json

@@ -1,6 +1,6 @@
 {
 	"name": "gamgee",
-	"version": "1.6.2",
+	"version": "1.6.3",
 	"description": "A Discord bot for managing a song request queue.",
 	"private": true,
 	"scripts": {

src/handleMessageComponent.ts

@@ -72,24 +72,24 @@ export async function handleMessageComponent(
 
 	switch (interaction.customId) {
 		case DONE_BUTTON.id:
-			logger.debug("Marking done....");
-			await markEntryDoneInQueue(message, queueChannel);
-			logger.debug("Marked an entry done.");
 			try {
 				await interaction.deferUpdate();
+				logger.debug("Marking done....");
+				await markEntryDoneInQueue(message, queueChannel);
+				logger.debug("Marked an entry done.");
 			} catch (error) {
-				logger.error(richErrorMessage(`Failed to defer update`, error));
+				logger.error(richErrorMessage(`Failed to mark done`, error));
 			}
 			break;
 
 		case RESTORE_BUTTON.id:
-			logger.debug("Marking undone....");
-			await markEntryNotDoneInQueue(message, queueChannel);
-			logger.debug("Marked an entry undone");
 			try {
 				await interaction.deferUpdate();
+				logger.debug("Marking undone....");
+				await markEntryNotDoneInQueue(message, queueChannel);
+				logger.debug("Marked an entry undone");
 			} catch (error) {
-				logger.error(richErrorMessage(`Failed to defer update`, error));
+				logger.error(richErrorMessage(`Failed to mark not done`, error));
 			}
 			break;