Merge pull request #1276 from mito-ds/default-color-to-none

mitosheet: fix bug with not supplying background_color

Author: aarondr77

mitosheet/mitosheet/tests/streamlit/test_streamlit.py

@@ -137,4 +137,31 @@ def test_spreadsheet_with_column_definitions():
         code_options={'as_function': True, 'call_function': False, 'function_name': 'test', 'function_params': {}}, 
         return_type='function'
     )
+    assert callable(f)
+
+@requires_streamlit   
+def test_spreadsheet_with_column_definitions_only_one_color():
+    f = spreadsheet(
+        df1, 
+        column_definitions=[
+            [
+                {
+                    'columns': ['A'],
+                    'conditional_formats': [{
+                        'filters': [{'condition': 'greater_than_or_equal', 'value': 5}], 
+                        'font_color': '#c30010', 
+                    }] 
+                },
+                {
+                    'columns': ['A'],
+                    'conditional_formats': [{
+                        'filters': [{'condition': 'less', 'value': 2}], 
+                        'background_color': '#ddcbd1' 
+                    }] 
+                }
+            ]
+        ], 
+        code_options={'as_function': True, 'call_function': False, 'function_name': 'test', 'function_params': {}}, 
+        return_type='function'
+    )
     assert callable(f)

mitosheet/mitosheet/utils.py

@@ -366,7 +366,7 @@ def get_default_df_formats(column_definitions: Optional[List[ColumnDefinitions]]
                     'filters': conditional_format['filters'],
                     'invalidFilterColumnIDs': [],
                     'color': font_color,
-                    'backgroundColor': conditional_format['background_color']
+                    'backgroundColor': background_color
                 }
 
                 conditional_formats.append(new_conditional_format)