Merge branch 'main' into update-issue-templates

.codespellrc

@@ -0,0 +1,3 @@
+[codespell]
+ignore-words-list= afterall
+skip = .git,*.svg,package-lock.json,node_modules,*lotties*,nwb-guide.spec,prepare_pyinstaller_spec.py

.github/workflows/chromatic.yml

@@ -2,8 +2,8 @@ name: Chromatic
 
 on:
   workflow_dispatch:
-  #pull_request_review:
-  #  types: [submitted]
+  pull_request_review:
+    types: [submitted]
 
 jobs:
   test:

.github/workflows/codespell.yml

@@ -0,0 +1,19 @@
+---
+name: Codespell
+
+on:
+  push:
+    branches: [main]
+  pull_request:
+    branches: [main]
+
+jobs:
+  codespell:
+    name: Check for spelling errors
+    runs-on: ubuntu-latest
+
+    steps:
+      - name: Checkout
+        uses: actions/checkout@v3
+      - name: Codespell
+        uses: codespell-project/actions-codespell@v1

prepare_pyinstaller_spec.py

@@ -40,4 +40,4 @@
 with open(file=Path(__file__).parent / "nwb-guide.spec", mode="w") as io:
     io.writelines(lines)
 
-print("Sucessfully injected recursion depth extension and json paths!")
+print("Successfully injected recursion depth extension and json paths!")

src/renderer/assets/css/global.css

@@ -12,7 +12,7 @@
     margin-top: 0px !important;
 }
 
-/*  Notfy */
+/*  Notyf */
 .notyf__toast {
     max-width: 40vw !important;
 }

src/renderer/assets/css/guided.css

@@ -1,6 +1,6 @@
 @import url("https://fonts.googleapis.com/css?family=Poppins:400,500,600,700");
 
-/* change the font to siblints of guided--body to Poppins except i elemenets */
+/* change the font to siblints of guided--body to Poppins except i elements */
 
 /*.guided--body *:not(i) {
   font-family: "Poppins", sans-serif;

src/renderer/src/stories/JSONSchemaInput.js

@@ -1090,7 +1090,7 @@ export class JSONSchemaInput extends LitElement {
 
                             if (schema.transform) newValue = schema.transform(newValue, this.value, schema);
 
-                            // // Do not check patter if value is empty
+                            // // Do not check pattern if value is empty
                             // if (schema.pattern && !isBlank) {
                             //     const regex = new RegExp(schema.pattern)
                             //     if (!regex.test(isNaN(newValue) ? value : newValue)) newValue = this.value // revert to last value

src/renderer/src/stories/pages/guided-mode/data/GuidedMetadata.js

@@ -228,7 +228,7 @@ export class GuidedMetadataPage extends ManagedPage {
 
             ignore: propsToIgnore,
             onOverride: (name) => {
-                this.notify(`<b>${header(name)}</b> has been overriden with a global value.`, "warning", 3000);
+                this.notify(`<b>${header(name)}</b> has been overridden with a global value.`, "warning", 3000);
             },
 
             transformErrors: (error) => {

src/renderer/src/stories/pages/guided-mode/data/GuidedSourceData.js

@@ -164,7 +164,7 @@ export class GuidedSourceDataPage extends ManagedPage {
             ignore: propsToIgnore,
             globals: this.info.globalState.project.SourceData,
             onOverride: (name) => {
-                this.notify(`<b>${header(name)}</b> has been overriden with a global value.`, "warning", 3000);
+                this.notify(`<b>${header(name)}</b> has been overridden with a global value.`, "warning", 3000);
             },
             // onlyRequired: true,
             onUpdate: () => (this.unsavedUpdates = "conversions"),

src/renderer/src/stories/pages/guided-mode/data/GuidedStructure.js

@@ -95,7 +95,7 @@ export class GuidedStructurePage extends Page {
             });
         }
 
-        await this.save(undefined, false); // Interrim save, in case the schema request fails
+        await this.save(undefined, false); // Interim save, in case the schema request fails
         await this.getSchema();
     };
 

src/renderer/src/stories/pages/guided-mode/options/GuidedUpload.js

@@ -105,7 +105,7 @@ export class GuidedUploadPage extends Page {
             if ("results" in globalUploadInfo) {
                 const result = await Swal.fire({
                     title: "This pipeline has already uploaded to DANDI",
-                    html: "Would you like to reupload the lastest files?",
+                    html: "Would you like to reupload the latest files?",
                     icon: "warning",
                     showCancelButton: true,
                     confirmButtonColor: "#3085d6",

src/renderer/src/stories/pages/guided-mode/setup/GuidedNewDatasetInfo.js

@@ -91,7 +91,7 @@ export class GuidedNewDatasetPage extends Page {
                 properties: ["createDirectory"],
             },
             onOverride: (name) => {
-                this.notify(`<b>${header(name)}</b> has been overriden with a global value.`, "warning", 3000);
+                this.notify(`<b>${header(name)}</b> has been overridden with a global value.`, "warning", 3000);
             },
             validateOnChange,
             onUpdate: () => (this.unsavedUpdates = true),

src/renderer/src/stories/pages/guided-mode/setup/GuidedSubjects.js

@@ -131,7 +131,7 @@ export class GuidedSubjectsPage extends Page {
             },
             onThrow: (message, type) => this.notify(message, type),
             onOverride: (name) => {
-                this.notify(`<b>${header(name)}</b> has been overriden with a global value.`, "warning", 3000);
+                this.notify(`<b>${header(name)}</b> has been overridden with a global value.`, "warning", 3000);
             },
             onUpdate: () => {
                 console.log("UPDATED!");

src/renderer/src/stories/preview/inspector/InspectorList.js

@@ -14,10 +14,10 @@ const sortList = (items) => {
             else return 1;
         })
         .sort((a, b) => {
-            const aLow = a.severity == "LOW";
-            const bLow = b.severity === "LOW";
-            if (aLow && bLow) return 0;
-            else if (aLow) return 1;
+            const lowA = a.severity == "LOW";
+            const lowB = b.severity === "LOW";
+            if (lowA && lowB) return 0;
+            else if (lowA) return 1;
             else return -1;
         });
 };

tests/e2e.test.ts

@@ -130,7 +130,7 @@ describe('E2E Test', () => {
         const dashboard = document.querySelector('nwb-dashboard')
         dashboard.sidebar.select('settings')
 
-        // Genereate test data
+        // Generate test data
         const page = dashboard.page
         page.deleteTestData()
         return await page.generateTestData()
@@ -319,7 +319,7 @@ describe('E2E Test', () => {
 
       await toNextPage('sourcedata')
 
-    })
+    }, 10 * 1000)
 
     test('Review source data information', async () => {
 
@@ -342,7 +342,7 @@ describe('E2E Test', () => {
 
       await toNextPage('inspect')
 
-    }, 20 * 1000) // Wait for conversion to complete
+    }, 30 * 1000) // Wait for conversion to complete
 
     test('Review NWB Inspector output', async () => {