Add support for null types when deriving JSON from Sass variables (#1019

) (#1020) * Fix collapsible nav build error Signed-off-by: Matt Provost <[email protected]> * Update to unset Signed-off-by: Matt Provost <[email protected]> * Fix in derive sass variable types Signed-off-by: Matt Provost <[email protected]> * Update changelog Signed-off-by: Matt Provost <[email protected]> --------- Signed-off-by: Matt Provost <[email protected]> (cherry picked from commit 8531d2a) Signed-off-by: github-actions[bot] <github-actions[bot]@users.noreply.github.com> # Conflicts: # CHANGELOG.md Co-authored-by: github-actions[bot] <github-actions[bot]@users.noreply.github.com>
opensearch-project · Sep 5, 2023 · 6535f9f · 6535f9f
1 parent 606d478
commit 6535f9f
Showing 1 changed file with 4 additions and 0 deletions.
diff --git a/scripts/derive-sass-variable-types.js b/scripts/derive-sass-variable-types.js
@@ -61,6 +61,10 @@ async function deriveSassVariableTypes(
 function deriveValueType(extractedValue) {
   switch (typeof extractedValue) {
     case 'object':
+      if (extractedValue === null) {
+        return ts.createKeywordTypeNode(ts.SyntaxKind.NullKeyword);
+      }
+
       return ts.createTypeLiteralNode(
         Object.keys(extractedValue).map(key =>
           ts.createPropertySignature(