Conversation
There was a problem hiding this comment.
Pull request overview
This PR enhances the UI by restructuring the audit display, improving visual consistency, and adding new functionality for USDM encounter generation.
Key changes:
- Moved audit sections from edit page to dedicated audits page
- Standardized action button styling and positioning across UI elements
- Added new utility functions for retrieving study metadata
- Created new module for generating USDM encounter objects from database
Reviewed changes
Copilot reviewed 12 out of 13 changed files in this pull request and generated 18 comments.
Show a summary per file
| File | Description |
|---|---|
| src/usdm/generate_encounters.py | New module to generate USDM encounter objects from database |
| src/soa_builder/web/utils.py | Added utility functions for retrieving API keys, SDTM values, and study metadata |
| src/soa_builder/web/templates/transition_rules_list.html | Restructured transition rule display with consistent action button styling |
| src/soa_builder/web/templates/index.html | Added Audits column linking to new audits page |
| src/soa_builder/web/templates/edit.html | Removed audit sections, standardized button styling, disabled some drag-and-drop features |
| src/soa_builder/web/templates/audits.html | New template displaying all audit trails in dedicated page |
| src/soa_builder/web/static/style.css | Added CSS for delete buttons, export buttons, and action containers |
| src/soa_builder/web/routers/visits.py | Added code generation for visit types and environmental settings |
| src/soa_builder/web/routers/audits.py | New router handling audit display endpoint |
| src/soa_builder/web/routers/arms.py | Fixed arm UID generation to always use max+1 instead of filling gaps |
| src/soa_builder/web/migrate_database.py | Added migration for new visit table columns |
| src/soa_builder/web/app.py | Integrated new audits router, moved utility functions, added inline comments |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
spelling Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
There was a problem hiding this comment.
Pull request overview
Copilot reviewed 12 out of 13 changed files in this pull request and generated 6 comments.
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
Clarification Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
There was a problem hiding this comment.
Pull request overview
Copilot reviewed 12 out of 13 changed files in this pull request and generated 1 comment.
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
No description provided.