vis/mail: fix include_picture typo and serialize bool params when false

novem/vis/grid.py

@@ -69,7 +69,7 @@ def __call__(self, content: Any, **kwargs: Any) -> Any:
         """
         Set's the layout of the grid
 
-        This paramter is expected to be one of the valid layout
+        This parameter is expected to be one of the valid layout
         strings or formats
         """
 

novem/vis/mail_sections.py

@@ -55,9 +55,8 @@ def __init__(
                     self._kwparams.append(ts)
 
             elif p.annotation is bool:
-                if v:
-                    ts = f"{k.replace('_', ' ')}: true"
-                    self._kwparams.append(ts)
+                ts = f"{k.replace('_', ' ')}: {'true' if v else 'false'}"
+                self._kwparams.append(ts)
 
             elif p.annotation is List[str]:
                 print("Treat as list of str")
@@ -184,7 +183,7 @@ def get_markdown(self) -> str:
 
         plist = self._params + self._kwparams + self._cparams
 
-        # Add section controleld params first
+        # Add section controlled params first
         for p in plist:
             if not len(p):
                 continue
@@ -413,7 +412,7 @@ def __init__(
         username: str,
         /,
         include_bio: bool = True,
-        include_pciture: bool = True,
+        include_picture: bool = True,
         override_bio: Optional[str] = None,
         **kwargs: str,
     ) -> None:

tests/test_mail_sections.py

@@ -70,6 +70,8 @@ def verify_put(key, val, request, context):
   ref: {plot_shortname}
   width: 100%
   align: center
+  include caption: false
+  include title: false
   include link: true
   override title: New title
   b: [l2 gray-300 purple, r1 red]
@@ -110,7 +112,7 @@ def test_mail_section_author(requests_mock):
     rest = """{{ author
   username: demo
   include bio: true
-  include pciture: true
+  include picture: true
 }}
 
 {{ /author }}"""