Merge pull request #17 from fkarg/fix-text-mentions

change previous mentions of `text` to `display`
krish-adi · Sep 22, 2023 · 04ddebb · 04ddebb
2 parents 91efb8c + 7124ebf
commit 04ddebb
Show file tree

Hide file tree

Showing 2 changed files with 2 additions and 2 deletions.
diff --git a/barfi/block_builder.py b/barfi/block_builder.py
@@ -128,7 +128,7 @@ def add_option(self, name: str, type: str, **kwargs) -> None:
 
         Interactive options interface:
             name (str)  : The name of the Option interface.
-            type (str)  : The type of the Option interface. 'checkbox', 'input', 'integer', 'number', 'select', 'slider', 'text'.
+            type (str)  : The type of the Option interface. 'checkbox', 'input', 'integer', 'number', 'select', 'slider', 'display'.
             value       : The default value for the option. Depends on the option chosen.
 
             Additional properties depending on the type of Option interface.

diff --git a/barfi/option_builder.py b/barfi/option_builder.py
@@ -123,7 +123,7 @@ def build_option(name: str, type: str, kwargs):
         option['type'] = "TextOption"
         value = kwargs.get('value', "null display")
         assert isinstance(
-            value, str), "Error: For text option, 'value' must be of type string."
+            value, str), "Error: For display option, 'value' must be of type string."
         option['value'] = value
 
     else: