Skip to content

Update home page isolate counts - #41

Merged
Ulthran merged 2 commits into
mainfrom
count-all-special-collections
Sep 24, 2025
Merged

Update home page isolate counts#41
Ulthran merged 2 commits into
mainfrom
count-all-special-collections

Conversation

@Ulthran

@Ulthran Ulthran commented Sep 24, 2025

Copy link
Copy Markdown
Contributor

No description provided.

Copilot AI review requested due to automatic review settings September 24, 2025 16:36

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull Request Overview

Updates the home page to dynamically display isolate counts for all special collections instead of hardcoding specific bacteremia and nares counts. The change makes the system more flexible by automatically showing counts for any special collection type present in the database.

  • Replaced hardcoded bacteremia and nares count queries with dynamic special collection counting
  • Added descriptive strings for known special collection types
  • Updated template to use a loop for displaying collection counts

Reviewed Changes

Copilot reviewed 2 out of 2 changed files in this pull request and generated 2 comments.

File Description
app/app.py Replaces hardcoded special collection queries with dynamic counting logic and description mapping
app/templates/index.html Updates template to loop through special collections instead of displaying fixed bacteremia/nares counts

Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.

Comment thread app/app.py
Comment thread app/app.py Outdated
@Ulthran
Ulthran merged commit 20455bb into main Sep 24, 2025
6 checks passed
@Ulthran
Ulthran deleted the count-all-special-collections branch September 24, 2025 17:11
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants