misc commentary

CHANGELOG.md

@@ -6,7 +6,7 @@ The format is based on [Keep a Changelog](https://keepachangelog.com/en/1.0.0/),
 and this project adheres to [Semantic Versioning](https://semver.org/spec/v2.0.0.html).
 
 
-## [0.3.18] - 2024.07.12
+## ?
 ### Fixed
 - List of offers wasn't showing.
 

README.md

@@ -39,6 +39,8 @@ npm run lint
 
 * Update the ClickUp tasks & CHANGELOG.md & the version in package.json, run `npm install`.
 
+* Commit everything (since the commit hash is used the app).
+
 * Record what version is currently on production.
 
 * Run the correct build:

playwright.config-local.ts

@@ -86,7 +86,7 @@ export default defineConfig({
    *     },
    *
    * But if we do then the testInfo.config.webServer is null and the API-setting test 00 fails.
-   * It is worth considering a change such that Time Safari's default Endorer API server is NOT set
+   * It is worth considering a change such that Time Safari's default Endorser API server is NOT set
    * in the user's settings so that it can be blanked out and the default is used.
    */
   webServer: {

src/registerServiceWorker.ts

@@ -2,6 +2,7 @@
 
 import { register } from "register-service-worker";
 
+// NODE_ENV is "production" by default with "vite build". See https://vitejs.dev/guide/env-and-mode
 if (import.meta.env.NODE_ENV === "production") {
   register("/sw_scripts-combined.js", {
     ready() {

vite.config.mjs

@@ -16,7 +16,7 @@ export default defineConfig({
       srcDir: '.',
       filename: 'sw_scripts-combined.js',
       manifest: {
-        // This is used for the app name. It doesn't include a space, because iOS complains if i recall correctly.
+        // This is used for the app name. It doesn't include a space, because iOS complains if I recall correctly.
         // There is a name with spaces in the constants/app.js file for use internally.
         name: process.env.TIME_SAFARI_APP_TITLE || require('./package.json').name,
         short_name: process.env.TIME_SAFARI_APP_TITLE || require('./package.json').name,