Skip to content

Add dynamic home page copy - #27

Merged
Ulthran merged 2 commits into
mainfrom
codex/add-copy-to-home-page
Aug 13, 2025
Merged

Add dynamic home page copy#27
Ulthran merged 2 commits into
mainfrom
codex/add-copy-to-home-page

Conversation

@Ulthran

@Ulthran Ulthran commented Aug 12, 2025

Copy link
Copy Markdown
Contributor

Summary

  • Update home page with detailed description and isolate statistics
  • Add database queries for isolate, patient, and sample counts

Testing

  • black .
  • pytest tests/ (fails: ModuleNotFoundError: No module named 'langchain_core.tracers._streaming')

https://chatgpt.com/codex/tasks/task_e_689bccc5fa98832397402d69acede45d

@Ulthran
Ulthran marked this pull request as ready for review August 13, 2025 16:01
Copilot AI review requested due to automatic review settings August 13, 2025 16:01

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

This PR updates the home page with a detailed description of the microbial ARC isolate database and adds dynamic statistics. The changes provide users with context about the database's purpose and current data volumes.

Key changes:

  • Replace static home page content with detailed description of the microbial ARC isolate database
  • Add database queries to display live counts of isolates, patients, bacteremia, and nares samples
  • Enhance query page with a help tooltip for the Custom Query field

Reviewed Changes

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

File Description
app/app.py Add database queries to fetch isolate, patient, bacteremia, and nares counts for the home page
app/templates/index.html Replace static content with detailed description and dynamic statistics display
app/templates/query.html Add informational popover tooltip to the Custom Query label
Comments suppressed due to low confidence (1)

app/templates/index.html:9

  • There is an unnecessary comma before the isolate count. It should be 'currently holds {{ isolate_count }} isolates' without the comma.
    Children’s Hospital of Philadelphia. The database currently holds, {{ isolate_count }} isolates from

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

@Ulthran
Ulthran merged commit 699783b into main Aug 13, 2025
1 of 6 checks passed
@Ulthran
Ulthran deleted the codex/add-copy-to-home-page branch August 13, 2025 16:37
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants