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

) * 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]>
opensearch-project · Sep 5, 2023 · 8531d2a · 8531d2a
1 parent 1230223
commit 8531d2a
Show file tree

Hide file tree

Showing 2 changed files with 5 additions and 0 deletions.
diff --git a/CHANGELOG.md b/CHANGELOG.md
@@ -28,6 +28,7 @@
 
 - Fix blurry text in breadcrumbs by avoiding skewing text ([#959](https://github.com/opensearch-project/oui/pull/959))
 - Remove `calc` usage from SchemaItem styles ([#990](https://github.com/opensearch-project/oui/pull/990))
+- Add support for null types when deriving JSON from Sass variables ([#1019](https://github.com/opensearch-project/oui/pull/1019))
 
 ### 🚞 Infrastructure
 

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(