Conversation
Changed the return type annotation of get_done_conversation_data from str to str | bool to reflect possible boolean return values.
Changed the return type of get_done_conversation_data to use GetConversationResponseModel for improved type safety. Also updated the import statement to include GetConversationResponseModel.
Split function signature and a long function call across multiple lines to improve code readability and maintainability. No functional changes were made.
Consolidated Streamlit session state variable initialization into a single loop for improved readability and maintainability. Removed redundant individual checks and assignments for each session state key.
Improved type annotations and default argument handling in patient queue functions in main.py. Added altair visualization library to requirements.txt.
This reverts commit 0c32e92.
…oxes, but one underneath another
Refactored the bed grid to group beds into rooms with improved tooltip handling and updated the CSS for better layout and hover effects. Added Polish translations for 'Unoccupied' and 'Room' in the locale file.
…into test-main
Updated the create_box_grid function to wrap all user-facing strings with the translation function (_), ensuring that room, bed, tooltips, and personnel fields are translatable. Also replaced st.write with st.markdown for rendering room HTML, and removed debug print statements.
Replaces unordered bed row info with an ordered, localized mapping for tooltip display in the box grid. Ensures consistent header order and translation, and formats personnel fields appropriately.
…into test-main
Test main
There was a problem hiding this comment.
Pull Request Overview
This PR enhances the frontend interface by introducing a room-based bed organization system and adding a new list view tab for better patient management visualization.
Key changes:
- Restructured bed display to group beds into rooms with a maximum of 3 beds per room
- Added a new "List view" tab that displays patients in a vertical list format showing patient names
- Improved session state initialization by consolidating multiple individual checks into a single loop
Reviewed Changes
Copilot reviewed 4 out of 5 changed files in this pull request and generated 4 comments.
| File | Description |
|---|---|
| frontend/main.py | Major refactoring to support room-based bed layout, added list view functionality, and improved session state management |
| frontend/locales/pl/LC_MESSAGES/base.po | Added Polish translations for new UI elements like "List view", "Room", and "Unoccupied" |
| frontend/agent.py | Enhanced type hints for conversation data handling functions |
| .env.sample | Added configuration for English voice agent ID |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
4 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Key changes
Minor changes