Separate data creation into reusable code chunk

Author: cgraham-rs

extensions/app-canary/app-canary.qmd

@@ -55,6 +55,7 @@ if show_instructions:
     results = []
     df = pd.DataFrame()  # Empty DataFrame
     check_time = datetime.datetime.now().strftime('%Y-%m-%d %H:%M:%S')
+    gt_tbl = None  # Define gt_tbl as None to avoid errors
 else:
     # Continue with normal execution
     # Headers for Connect API
@@ -147,30 +148,11 @@ else:
     check_time = datetime.datetime.now().strftime('%Y-%m-%d %H:%M:%S')
 ```
 
-::: {.card .table-card .content-visible when-format="dashboard"}
 ```{python}
 #| echo: false
 
-# Display instructions if setup failed
-if show_instructions:
-    from IPython.display import display, HTML
-    html_content = """
-    <h2>Setup Instructions</h2>
-    <ul>
-    """
-    for instruction in instructions:
-        html_content += f"<li>{instruction}</li>"
-    
-    html_content += """
-    </ul>
-    <p>Posit Connect documentation for Vars (environment variables): <a href="https://docs.posit.co/connect/user/content-settings/#content-vars" target="_blank">https://docs.posit.co/connect/user/content-settings/#content-vars</a></p>
-    """
-    display(HTML(html_content))
-
-
-# Display table with basic styling
+# Create a shared table that will be used by both dashboard and email formats
 if not show_instructions and not df.empty:
-    
     # First create links for name and guid columns
     df_display = df.copy()
     
@@ -199,24 +181,51 @@ if not show_instructions and not df.empty:
                   locations=loc.body(columns="status", rows=lambda df: df["status"] == "FAIL")
               ))
     
-    # Display the table
-    gt_tbl
-```
-:::
+    # Create HTML version with inline CSS for email
+    gt_html = gt_tbl.as_raw_html(inline_css=True)
+else:
+    gt_tbl = None
+    gt_html = "<p>No applications are currently being monitored.</p>"
 
+# Compute if we should send an email, only send if at least one app has a failure
+send_email = not df.empty and (df['status'] == 'FAIL').any() if 'df' in locals() and 'status' in df.columns else False
+```
 
+::: {.card .table-card .content-visible when-format="dashboard"}
 ```{python}
 #| echo: false
 
-# Computer if we should send an email, only send if at least one app has a failure
-send_email = not df.empty and (df['status'] == 'FAIL').any()
+# Display instructions if setup failed
+if show_instructions:
+    from IPython.display import display, HTML
+    html_content = """
+    <h2>Setup Instructions</h2>
+    <ul>
+    """
+    for instruction in instructions:
+        html_content += f"<li>{instruction}</li>"
+    
+    html_content += """
+    </ul>
+    <p>Posit Connect documentation for Vars (environment variables): <a href="https://docs.posit.co/connect/user/content-settings/#content-vars" target="_blank">https://docs.posit.co/connect/user/content-settings/#content-vars</a></p>
+    """
+    display(HTML(html_content))
+
+# Display the table (now using the pre-created gt_tbl from above)
+elif gt_tbl is not None:
+    gt_tbl
+else:
+    display(HTML("<p>No applications are currently being monitored.</p>"))
 ```
+:::
 
 
 ::: {.email .content-visible when-format="email"}
 
 ::: {.email-scheduled}
-`{python} send_email`
+```{python}
+send_email
+```
 :::
 
 ::: {.subject}
@@ -234,8 +243,7 @@ One or more of the apps you are monitoring have failed.
 ```{python}
 #| echo: false
 
-HTML(gt_tbl.as_raw_html(inline_css=True))
+# Use the pre-generated HTML with inline CSS
+HTML(gt_html)
 ```
-
-
 :::