Add default for optional additional_segments parameter in video con…

…sent (#89) * fix undefined default for optional additional_segments param, and move text property into nested param object * fix formatting * add changeset
lookit · Oct 29, 2024 · 7230f51 · 7230f51
1 parent de38876
commit 7230f51
Show file tree

Hide file tree

Showing 2 changed files with 11 additions and 4 deletions.
diff --git a/.changeset/smart-books-scream.md b/.changeset/smart-books-scream.md
@@ -0,0 +1,6 @@
+---
+"@lookit/record": patch
+---
+
+Add missing default value for video consent "additional_segments" parameter,
+which was causing an error when this optional parameter was omitted (#84).
diff --git a/packages/record/src/consentVideo.ts b/packages/record/src/consentVideo.ts
@@ -46,15 +46,16 @@ const info = <const>{
     additional_segments: {
       type: ParameterType.COMPLEX,
       array: true,
+      default: [],
       nested: {
         title: {
           type: ParameterType.STRING,
           default: "",
         },
-      },
-      text: {
-        type: ParameterType.STRING,
-        default: "",
+        text: {
+          type: ParameterType.STRING,
+          default: "",
+        },
       },
     },
     prompt_all_adults: { type: ParameterType.BOOL, default: false },