docassemble.AssemblyLine 4.3.0__tar.gz → 4.5.0__tar.gz

{docassemble_assemblyline-4.3.0/docassemble.AssemblyLine.egg-info → docassemble_assemblyline-4.5.0}/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.4
 Name: docassemble.AssemblyLine
-Version: 4.3.0
+Version: 4.5.0
 Summary: Quickly go from a paper court form to a runnable, guided, step-by-step web application powered by Docassemble. Swap out branding and pre-built questions to meet your needs.
 Home-page: https://courtformsonline.org
 Author: Suffolk Legal Innovation and Technology Lab
@@ -9,7 +9,7 @@ License-Expression: MIT
 Project-URL: Homepage, https://courtformsonline.org
 Description-Content-Type: text/markdown
 License-File: LICENSE
-Requires-Dist: docassemble.ALToolbox>=0.12.0
+Requires-Dist: docassemble.ALToolbox>=0.16.0
 Requires-Dist: docassemble.GitHubFeedbackForm>=0.4.1.1
 Dynamic: author
 Dynamic: home-page

docassemble_assemblyline-4.5.0/docassemble/AssemblyLine/__init__.py

@@ -0,0 +1 @@
+__version__ = '4.5.0'

{docassemble_assemblyline-4.3.0 → docassemble_assemblyline-4.5.0}/docassemble/AssemblyLine/al_courts.py

@@ -2,13 +2,11 @@
 Package for a very simple / MVP list of courts that is mostly signature compatible w/ MACourts for now
 """
 
-import os
-from typing import Any, Callable, Dict, List, Mapping, Optional, Union, Set, Tuple
+from typing import Any, Callable, Dict, List, Optional, Union, Set, Tuple
 import pandas as pd
 import docassemble.base.functions
 from docassemble.base.util import (
     path_and_mimetype,
-    Address,
     LatitudeLongitude,
     DAObject,
     log,

{docassemble_assemblyline-4.3.0 → docassemble_assemblyline-4.5.0}/docassemble/AssemblyLine/al_document.py

@@ -35,7 +35,6 @@ from docassemble.base.pdfa import pdf_to_pdfa
 from textwrap import wrap
 from math import floor
 import subprocess
-from collections import ChainMap
 import pikepdf
 from typing import Tuple
 import secrets
@@ -1582,6 +1581,7 @@ class ALDocumentBundle(DAList):
             self.auto_gather = False
         if "gathered" not in kwargs:
             self.gathered = True
+        self._set_default_attributes()
         self.initializeAttribute("cache", DALazyAttribute)
         self.always_enabled = hasattr(self, "enabled") and self.enabled
         # Pre-cache some DALazyTemplates we set up to aid translation that won't
@@ -1590,6 +1590,24 @@ class ALDocumentBundle(DAList):
             # When the key is "preview", append it to the file name
             self.suffix_to_append = "preview"
 
+    def _set_default_attributes(self) -> None:
+        if not hasattr(self, "add_page_numbers"):
+            self.add_page_numbers = False
+        if not hasattr(self, "page_number_prefix"):
+            self.page_number_prefix = ""
+        if not hasattr(self, "page_number_start"):
+            self.page_number_start = 1
+        if not hasattr(self, "page_number_digits"):
+            self.page_number_digits = 5
+        if not hasattr(self, "page_number_area"):
+            self.page_number_area = None
+        if not hasattr(self, "page_number_font_size"):
+            self.page_number_font_size = 10
+        if not hasattr(self, "page_number_offset_horizontal"):
+            self.page_number_offset_horizontal = 15
+        if not hasattr(self, "page_number_offset_vertical"):
+            self.page_number_offset_vertical = 15
+
     def as_pdf(
         self,
         key: str = "final",
@@ -1649,6 +1667,17 @@ class ALDocumentBundle(DAList):
                 filename=f"{base_name(self.filename)}{append_suffix}.pdf",
                 pdfa=pdfa,
             )
+        if hasattr(self, "add_page_numbers") and self.add_page_numbers:
+            self._set_default_attributes()
+            pdf.bates_number(
+                prefix=self.page_number_prefix,
+                start=self.page_number_start,
+                digits=self.page_number_digits,
+                area=self.page_number_area,
+                font_size=self.page_number_font_size,
+                offset_horizontal=self.page_number_offset_horizontal,
+                offset_vertical=self.page_number_offset_vertical,
+            )
         pdf.title = self.title
         setattr(self.cache, safe_key, pdf)
 
@@ -2386,6 +2415,7 @@ class ALDocumentBundle(DAList):
         label: str = "Send",
         icon: str = "envelope",
         preferred_formats: Optional[Union[str, List[str]]] = None,
+        email_legend_class: str = "h4",
     ) -> str:
         """
         Generate HTML for an input box and button that allows someone to send the bundle
@@ -2405,6 +2435,8 @@ class ALDocumentBundle(DAList):
             icon (str, optional): The Fontawesome icon for the button. Defaults
                 to "envelope".
             preferred_formats (Optional[Union[str,List[str]]], optional): A list of allowed formats for the document. Defaults to "pdf" if not specified.
+            email_legend_class (str, optional): CSS class applied to the email
+                section legend. Defaults to "h4".
 
         Returns:
             str: The generated HTML string for the input box and button.
@@ -2444,7 +2476,7 @@ class ALDocumentBundle(DAList):
         # Container of whole email section with header
         return_str = f"""
   <fieldset class="al_send_bundle al_send_section_alone {html_safe_str(self.instanceName)}" id="al_send_bundle_{name}" name="al_send_bundle_{name}">
-    <legend class="h4 al_doc_email_header">{str(self.get_email_copy)}</legend> 
+    <legend class="{email_legend_class} al_doc_email_header">{str(self.get_email_copy)}</legend> 
     """
         # "Editable" checkbox
         if (
@@ -2736,9 +2768,14 @@ class ALExhibit(DAObject):
         self,
         *,
         refresh: bool = False,
-        prefix: str = "",
         pdfa: bool = False,
         add_page_numbers: bool = True,
+        page_number_prefix: str = "",
+        page_number_digits: int = 5,
+        page_number_area=None,
+        page_number_font_size: float = 10,
+        page_number_offset_horizontal: float = 15,
+        page_number_offset_vertical: float = 15,
         add_cover_page: bool = True,
         filename: Optional[str] = None,
         append_matching_suffix: bool = True,
@@ -2750,9 +2787,14 @@ class ALExhibit(DAObject):
 
         Args:
             refresh (bool): If True, forces the exhibit to refresh before generating the PDF. (unused, provided for signature compatibility)
-            prefix (str): Prefix for Bates numbering if 'add_page_numbers' is True.
             pdfa (bool): If True, the generated PDF will be in PDF/A format.
             add_page_numbers (bool): If True, apply Bates numbering starting from 'self.start_page'.
+            page_number_prefix (str): If add_page_numbers is True, this gets added to the beginning on the bates number each page (e.g. `EX-`).
+            page_number_digits (int): How many digits (i.e. leading 0s) that the bates number will have
+            page_number_area (str): Where on the page the bates number will go ("TOP_LEFT", "TOP_RIGHT", "BOTTOM_LEFT", "BOTTOM_RIGHT" (default))
+            page_number_font_size (float): How big the bates page number will be in points (default is 10).
+            page_number_offset_horizontal (float): The number of pixels that the bates page number is offset from the left / right of the page.
+            page_number_offset_vertical (float): The number of pixels that the bates page number is offset from the top / bottom of the page.
             add_cover_page (bool): If True, prepend the exhibit with a cover page.
             filename (Optional[str]): Custom filename for the generated PDF. Default is "exhibits.pdf".
             append_matching_suffix (bool): If True, appends a suffix to the filename based on certain matching criteria.
@@ -2780,7 +2822,15 @@ class ALExhibit(DAObject):
             )
 
         if add_page_numbers:
-            concatenated_pages.bates_number(prefix=prefix, start=self.start_page)
+            concatenated_pages.bates_number(
+                start=self.start_page,
+                prefix=page_number_prefix,
+                digits=page_number_digits,
+                area=page_number_area,
+                font_size=page_number_font_size,
+                offset_horizontal=page_number_offset_horizontal,
+                offset_vertical=page_number_offset_vertical,
+            )
 
         setattr(self._cache, safe_key, concatenated_pages)
         return getattr(self._cache, safe_key)
@@ -2914,6 +2964,12 @@ class ALExhibitList(DAList):
         filename="file.pdf",
         pdfa: bool = False,
         add_page_numbers: bool = False,
+        page_number_prefix: str = "",
+        page_number_digits: int = 5,
+        page_number_area=None,
+        page_number_font_size: float = 10,
+        page_number_offset_horizontal: float = 15,
+        page_number_offset_vertical: float = 15,
         toc_pages: int = 0,
         append_matching_suffix: bool = True,
     ) -> DAFile:
@@ -2924,6 +2980,12 @@ class ALExhibitList(DAList):
             filename (str): Desired filename for the generated PDF.
             pdfa (bool): If True, generates the PDF in PDF/A format.
             add_page_numbers (bool): If True, adds page numbers to the generated PDF.
+            page_number_prefix (str): What the bates number added to each page should start with (e.g. `EX-`).
+            page_number_digits (int): How many digits (i.e. leading 0s) that the bates number will have
+            page_number_area (str): Where on the page the bates number will go ("TOP_LEFT", "TOP_RIGHT", "BOTTOM_LEFT", "BOTTOM_RIGHT" (default))
+            page_number_font_size (float): How big the bates page number will be in points (default is 10).
+            page_number_offset_horizontal (float): The number of pixels that the bates page number is offset from the left / right of the page.
+            page_number_offset_vertical (float): The number of pixels that the bates page number is offset from the top / bottom of the page.
             toc_pages (int): Expected number of pages in the table of contents.
             append_matching_suffix (bool): If True, appends matching suffix to the filename.
 
@@ -2935,12 +2997,19 @@ class ALExhibitList(DAList):
                 exhibit.cover_page
         if self.include_table_of_contents and toc_pages != 1:
             self._update_page_numbers(toc_guess_pages=toc_pages)
+        if not page_number_prefix and self.bates_prefix:
+            page_number_prefix = self.bates_prefix
         return pdf_concatenate(
             [
                 exhibit.as_pdf(
                     add_cover_page=self.include_exhibit_cover_pages,
                     add_page_numbers=add_page_numbers,
-                    prefix=self.bates_prefix,
+                    page_number_prefix=page_number_prefix,
+                    page_number_digits=page_number_digits,
+                    page_number_area=page_number_area,
+                    page_number_font_size=page_number_font_size,
+                    page_number_offset_horizontal=page_number_offset_horizontal,
+                    page_number_offset_vertical=page_number_offset_vertical,
                 )
                 for exhibit in self
             ],
@@ -3073,12 +3142,19 @@ class ALExhibitDocument(ALDocument):
     has_addendum: bool
     auto_labeler: Callable
     auto_ocr: bool
+    # Deprecated
     bates_prefix: str
     maximum_size: int
     maximum_size_per_doc: int
     suffix_to_append: str
     exhibits: ALExhibitList
     table_of_contents: DAFile
+    page_number_prefix: str
+    page_number_digits: int
+    page_number_area: str | None
+    page_number_font_size: float
+    page_number_offset_horizontal: float
+    page_number_offset_vertical: float
 
     def init(self, *pargs, **kwargs) -> None:
         """Standard DAObject init method.
@@ -3093,8 +3169,6 @@ class ALExhibitDocument(ALDocument):
             self.exhibits.auto_labeler = self.auto_labeler
         if hasattr(self, "auto_ocr"):
             self.exhibits.auto_ocr = self.auto_ocr
-        if hasattr(self, "bates_prefix"):
-            self.exhibits.bates_prefix = self.bates_prefix
         if hasattr(self, "include_exhibit_cover_pages"):
             self.exhibits.include_exhibit_cover_pages = self.include_exhibit_cover_pages
         else:
@@ -3112,13 +3186,30 @@ class ALExhibitDocument(ALDocument):
         else:
             self.include_table_of_contents = True
             self.exhibits.include_table_of_contents = True
+        if hasattr(self, "bates_prefix"):
+            self.exhibits.bates_prefix = self.bates_prefix
         if not hasattr(self, "add_page_numbers"):
             self.add_page_numbers = False
+        self._set_default_attributes()
         self.has_addendum = False
         if not hasattr(self, "suffix_to_append"):
             # When the key is "preview", append it to the file name
             self.suffix_to_append = "preview"
 
+    def _set_default_attributes(self) -> None:
+        if not hasattr(self, "page_number_prefix"):
+            self.page_number_prefix = ""
+        if not hasattr(self, "page_number_digits"):
+            self.page_number_digits = 5
+        if not hasattr(self, "page_number_area"):
+            self.page_number_area = None
+        if not hasattr(self, "page_number_font_size"):
+            self.page_number_font_size = 10
+        if not hasattr(self, "page_number_offset_horizontal"):
+            self.page_number_offset_horizontal = 15
+        if not hasattr(self, "page_number_offset_vertical"):
+            self.page_number_offset_vertical = 15
+
     def has_overflow(self) -> bool:
         """
         Check if there is any overflow in the document.
@@ -3193,12 +3284,21 @@ class ALExhibitDocument(ALDocument):
             filename = base_name(self.filename) + ".pdf"
 
         if len(self.exhibits):
+            self._set_default_attributes()
             if self.include_table_of_contents:
                 toc_pages = self.table_of_contents.num_pages()
                 return pdf_concatenate(
                     self.table_of_contents,
                     self.exhibits.as_pdf(
-                        add_page_numbers=self.add_page_numbers, toc_pages=toc_pages
+                        add_page_numbers=self.add_page_numbers,
+                        page_number_prefix=self.page_number_prefix,
+                        page_number_digits=self.page_number_digits,
+                        page_number_area=self.page_number_area,
+                        page_number_font_size=self.page_number_font_size,
+                        page_number_offset_horizontal=self.page_number_offset_horizontal,
+                        page_number_offset_vertical=self.page_number_offset_vertical,
+                        toc_pages=toc_pages,
+                        pdfa=pdfa,
                     ),
                     filename=filename,
                     pdfa=pdfa,
@@ -3206,6 +3306,12 @@ class ALExhibitDocument(ALDocument):
             else:
                 return self.exhibits.as_pdf(
                     add_page_numbers=self.add_page_numbers,
+                    page_number_prefix=self.page_number_prefix,
+                    page_number_digits=self.page_number_digits,
+                    page_number_area=self.page_number_area,
+                    page_number_font_size=self.page_number_font_size,
+                    page_number_offset_horizontal=self.page_number_offset_horizontal,
+                    page_number_offset_vertical=self.page_number_offset_vertical,
                     filename=filename,
                     pdfa=pdfa,
                 )

{docassemble_assemblyline-4.3.0 → docassemble_assemblyline-4.5.0}/docassemble/AssemblyLine/al_general.py

@@ -30,13 +30,11 @@ from docassemble.base.util import (
     their,
     url_action,
     validation_error,
-    value,
     word,
     your,
     log,
     value,
 )
-from docassemble.ALToolbox.misc import fa_icon
 import docassemble.base.functions
 import random
 import re
@@ -325,7 +323,7 @@ class ALAddress(Address):
             if not bare and (
                 unit_lower.isnumeric()
                 or (
-                    not " " in self.unit
+                    " " not in self.unit
                     and not any(
                         x in unit_lower
                         for x in [
@@ -1660,10 +1658,13 @@ class ALIndividual(Individual):
         Used to assist with checkbox filling in PDFs with "skip undefined"
         turned on.
         """
-        return not (
-            self.gender
-            in ["prefer-not-to-say", "male", "female", "unknown", "nonbinary"]
-        )
+        return self.gender not in [
+            "prefer-not-to-say",
+            "male",
+            "female",
+            "unknown",
+            "nonbinary",
+        ]
 
     def contact_fields(self) -> None:
         """
@@ -1913,7 +1914,7 @@ class ALIndividual(Individual):
             if len(pronouns_to_use) > 0:
                 output = "/".join(pronouns_to_use)
             else:
-                output = default
+                output = f"{default}'s {target}"
         elif hasattr(self, "person_type") and self.person_type in [
             "business",
             "organization",
@@ -1927,7 +1928,7 @@ class ALIndividual(Individual):
             else:
                 output = their(target, **kwargs)
         else:
-            output = default
+            output = f"{default}'s {target}"
 
         if "capitalize" in kwargs and kwargs["capitalize"]:
             return capitalize(output)

{docassemble_assemblyline-4.3.0 → docassemble_assemblyline-4.5.0}/docassemble/AssemblyLine/custom_jinja_filters.py

@@ -5,9 +5,8 @@
 
 # See: https://docassemble.org/docs/documents.html#register_jinja_filter
 
-import re
 from typing import Any, Dict, List, Optional, Union
-from docassemble.base.util import register_jinja_filter, DACatchAll, word
+from docassemble.base.util import register_jinja_filter, DACatchAll
 from jinja2 import Undefined, pass_context
 from jinja2.runtime import Context as Jinja2Context
 
@@ -18,6 +17,7 @@ __all__ = [
     "catchall_question",
     "catchall_subquestion",
     "catchall_fields_code",
+    "catchall_complete",
 ]
 
 
@@ -29,6 +29,7 @@ def catchall_options(value: Any, *raw_items: Any) -> DACatchAll:
 
     The items can be in various formats:
 
+    - String: `"code_is_the_label"`
     - String: `"code: label"`
     - Dictionary: `{"code": "label"}`
     - Tuple: `("code", "label")`
@@ -66,14 +67,14 @@ def catchall_options(value: Any, *raw_items: Any) -> DACatchAll:
         DACatchAll: The modified DACatchAll object with the assigned options.
     """
     if isinstance(value, DACatchAll):
-        pairs = []
+        pairs: list[str | tuple[str, str]] = []
         for item in raw_items:
             if isinstance(item, str):
                 if ":" in item:
                     code, label = item.split(":", 1)
+                    pairs.append((code.strip(), label.strip()))
                 else:
-                    code = label = item
-                pairs.append((code.strip(), label.strip()))
+                    pairs.append(item.strip())
             elif isinstance(item, dict):
                 for code, label in item.items():
                     pairs.append((code.strip(), label.strip()))
@@ -203,6 +204,49 @@ def catchall_subquestion(value: Any, subquestion: str) -> DACatchAll:
     return value
 
 
+def catchall_complete(
+    value: DACatchAll,
+    question: str = "",
+    subquestion: str = "",
+    label: str = "",
+    datatype: str = "",
+    options: Any = "",
+) -> DACatchAll:
+    """
+    Jinja2 filter to allow you to define multiple attributes of a DACatchAll field inside a DOCX template.
+
+    Each argument after `value` corresponds to the similarly named `catchall_` function.
+
+    Example usage in a DOCX template:
+    ```
+    {{ my_catchall_field | catchall_complete(question="What additional information do you need?", subquestion="Be specific", label="no label") }}
+    ```
+
+    Args:
+        value (DACatchAll): The DACatchAll object to which the attributes will be assigned.
+        question (str, optional): The question text to set on the DACatchAll object.
+        subquestion (str, optional): The subquestion text to set on the DACatchAll object.
+        label (str, optional): The label to set on the DACatchAll object.
+        datatype (str, optional): The datatype to set on the DACatchAll object.
+        options (Any, optional): The options to set on the DACatchAll object.
+
+    Returns:
+        DACatchAll: The modified DACatchAll object with the assigned attributes.
+    """
+    if isinstance(value, DACatchAll):
+        if question:
+            catchall_question(value, question)
+        if subquestion:
+            catchall_subquestion(value, subquestion)
+        if label:
+            catchall_label(value, label)
+        if datatype:
+            catchall_datatype(value, datatype)
+        if options:
+            catchall_options(value, options)
+    return value
+
+
 def _undefined_label(ud: Undefined) -> Union[str, None]:
     """
     Extract the variable name from a Jinja2 Undefined object for use as a placeholder label.
@@ -314,6 +358,7 @@ register_jinja_filter("catchall_label", catchall_label)
 register_jinja_filter("catchall_datatype", catchall_datatype)
 register_jinja_filter("catchall_question", catchall_question)
 register_jinja_filter("catchall_subquestion", catchall_subquestion)
+register_jinja_filter("catchall_complete", catchall_complete)
 register_jinja_filter("if_final", if_final)
 
 

{docassemble_assemblyline-4.3.0 → docassemble_assemblyline-4.5.0}/docassemble/AssemblyLine/data/questions/al_code.yml

@@ -22,14 +22,14 @@ code: |
   form_delivery_complete = True
 ---
 code: |
-  court_email = court_emails.get(trial_court.name,None)
+  court_email = court_emails.get(trial_court.name, None)
   if not court_email:
     found_email = False
   else:
     found_email = True
 ---
 code: |
-  bcc_failsafe = court_emails.get('Failsafe Address',None)
+  bcc_failsafe = court_emails.get("Failsafe Address", None)
 ---
 comment: |
   You are encouraged to overwrite these emails during testing!
@@ -51,22 +51,50 @@ code: |
   else:
     email_to_use = dev_email_to_use
     bcc_email = dev_bcc_email
-  
+
   # temporary for testing
   # email_to_use = "massaccess@suffolk.edu"
 
-  if task_not_yet_performed('send email'):
+  if task_not_yet_performed("send email"):
     log(f"Court email is {court_email}, sending email to {email_to_use}")
     email_success = False
     if should_cc_user:
-      log('Sending email to ' + email_to_use + ' and ccing ' + cc_email + ' for ' + str(users) + ' bcc to ' + bcc_email)
-      email_success = send_email(to=email_to_use, template=email_to_court_template, task='send email', 
-                                 cc=cc_email, bcc=bcc_email, attachments=al_final_form_to_file.as_pdf_list(key='final'))
+      log(
+        "Sending email to "
+        + email_to_use
+        + " and ccing "
+        + cc_email
+        + " for "
+        + str(users)
+        + " bcc to "
+        + bcc_email
+      )
+      email_success = send_email(
+        to=email_to_use,
+        template=email_to_court_template,
+        task="send email",
+        cc=cc_email,
+        bcc=bcc_email,
+        attachments=al_final_form_to_file.as_pdf_list(key="final"),
+      )
     else:
-      log('Sending email to ' + email_to_use + ' and no cc ' + ' for ' + str(users) + ' bcc to ' + bcc_email)
-      email_success = send_email(to=email_to_use, template=email_to_court_template, task='send email', 
-                                 attachments=al_final_form_to_file.as_pdf_list(key='final'), bcc=bcc_email)
-    mark_task_as_performed('send email')
+      log(
+        "Sending email to "
+        + email_to_use
+        + " and no cc "
+        + " for "
+        + str(users)
+        + " bcc to "
+        + bcc_email
+      )
+      email_success = send_email(
+        to=email_to_use,
+        template=email_to_court_template,
+        task="send email",
+        attachments=al_final_form_to_file.as_pdf_list(key="final"),
+        bcc=bcc_email,
+      )
+    mark_task_as_performed("send email")
   sent_email_to_court = True
 ---
 ##############################################################
@@ -90,7 +118,7 @@ code: |
   # Gets the number of pages, plus the first cover page
   # TODO(brycew): figure out a better way to estimate the number of pages,
   # including addendum, without generating a full file
-  al_num_package_pages = al_court_bundle.as_pdf(key='preview').num_pages() + 1
+  al_num_package_pages = al_court_bundle.as_pdf(key="preview").num_pages() + 1
 ---
 code: |
   # You can define this in the interview's code block.
@@ -101,7 +129,7 @@ code: |
   # This is a fallback package title to avoid errors on new forms
   # It defines package_title as the metadata title.
   # You may want a better form title for most interviews
-  package_title = all_variables(special='metadata').get('title','').rstrip()
+  package_title = all_variables(special="metadata").get("title", "").rstrip()
 ---
 attachment:
   variable name: al_cover_page[i]
@@ -126,27 +154,36 @@ code: |
   elif signature_choice == "typed_signature":
     for signature in signature_fields:
       # Signature fields are strings like "users[0].signature". If the string ends in .signature, remove that and replace with .typed_signature, only at the end of the string
-      if signature.endswith('signature'):
-        typed_signature = signature[:-9] + 'typed_signature'
+      if signature.endswith("signature"):
+        typed_signature = signature[:-9] + "typed_signature"
         define(signature, DAFile(signature, filename="signature.png"))
         value(signature).initialize()
-        create_signature(value(typed_signature), value(signature).path(), signature_prefix = al_typed_signature_prefix, font_name = al_typed_signature_font)
+        create_signature(
+          value(typed_signature),
+          value(signature).path(),
+          signature_prefix=al_typed_signature_prefix,
+          font_name=al_typed_signature_font,
+        )
       else:
-        log("Signature field " + signature + " does not end with .signature, not creating typed signature")
-  elif signature_choice in ['this device', 'this_device']:
+        log(
+          "Signature field "
+          + signature
+          + " does not end with .signature, not creating typed signature"
+        )
+  elif signature_choice in ["this device", "this_device"]:
     for signature in signature_fields:
-      value(signature)    
+      value(signature)
   # Check for Branch 2 or 3
-  elif signature_choice == 'phone':
+  elif signature_choice == "phone":
     saw_signature_qrcode
     # User will click next. link_cell will only be defined if they chose the texting option
-    if showifdef("link_cell") and task_not_yet_performed('send signature link'): 
+    if showifdef("link_cell") and task_not_yet_performed("send signature link"):
       # They used the text option
-      send_sms(task='send signature link', to=link_cell,template=interview_link)
+      send_sms(task="send signature link", to=link_cell, template=interview_link)
       signature_wait_screen
       for signature in signature_fields:
         value(signature)
-    else: # Branch 3: They used the QR Code. No special screen, just continue
+    else:  # Branch 3: They used the QR Code. No special screen, just continue
       for signature in signature_fields:
         value(signature)
     # Show the follow-up if they sent the link to their phone by QR or by SMS
@@ -157,14 +194,14 @@ code: |
 code: |
   # Sometimes we have values that can be set from interview_metadata.
   # They don't always work, but interview author can override.
-  
+
   # Some interviews have multiple interview_metadata keys. Look in the 1st one
-  first_form_key = next(iter(interview_metadata),'')
-  if isinstance(interview_metadata.get(first_form_key,{}), dict):
-    first_form = interview_metadata.get(first_form_key,{})
+  first_form_key = next(iter(interview_metadata), "")
+  if isinstance(interview_metadata.get(first_form_key, {}), dict):
+    first_form = interview_metadata.get(first_form_key, {})
   else:
     first_form = {}
-  del(first_form_key)
+  del first_form_key
 ---
 depends on:
   - user_ask_role
@@ -175,12 +212,12 @@ code: |
   # define user_role to be either "defendant" or "plaintiff",
   # either by checking interview_metadata or asking the user.
   # The only valid values for user_role are "plaintiff" and "defendant".
-  
+
   # We will check for the "typical_role" key.
   # Some older wizarded code doesn't define `typical_role`, so fall back
   # to "unknown" if the key is not present.
   # first_form is the first entry in interview_metadata
-  if not first_form.get("typical_role", 'unknown') == 'unknown':
+  if not first_form.get("typical_role", "unknown") == "unknown":
     user_role = first_form.get("typical_role")
   else:
     # If the user role was set to "unknown" at the time wizard run