feat: Hospital Learning & Patient Management Academy Initial (Mobile) by SB2318 · Pull Request #2111 · SB2318/UltimateHealth

SB2318 · 2026-07-11T16:48:00Z

PR Description

This pull request introduces the initial mobile implementation of the Hospital Learning & Patient Management Academy for UltimateHealth.

The implementation establishes the foundational architecture for the Academy module, providing users with a dedicated learning experience focused on hospital operations and patient management. This PR focuses on the frontend structure and reusable UI components, allowing future issues to build interactive learning features on top of this foundation.

Changes Included

Added the Academy module to the mobile application.
Integrated Academy into the application navigation.
Implemented the Academy Dashboard.
Added Course Listing screen with reusable course cards.
Added Course Details screen.
Implemented Lesson Reader UI.
Added reusable components for:
- Academy Cards
- Course Cards
- Lesson Cards
- Progress Cards
- Category Chips
- Continue Learning section
Added mock data for initial development.
Implemented responsive layouts.
Added dark mode compatibility.
Improved overall code organization for future Academy features.

This implementation serves as the foundation for upcoming features including quizzes, workflow visualizations, patient journey simulations, certificates, gamification, AI-assisted learning, and backend integration.

Type of Change

Bug fix (change which fixes an issue)
New feature (change which adds functionality)
Documentation update

Select your work-area

Frontend
Backend
Documentation
Others

Related Issue

Implements the initial mobile version of the Hospital Learning & Patient Management Academy.

Add your Work Example

Academy Dashboard
Course Listing
Course Details
Lesson Reader
Responsive layouts
Dark mode (if applicable)

Fixes (mention the issue number which this fixes)

#closes #2110

Checklist

I have updated my branch and synced it with the project's 'develop' branch before making this PR.
I have optimized the file changes.
I have added a snapshot of my work example.
I have made a PR to the project's develop branch.

Undertaking

My code follows the style guidelines of this project.
I have performed a self-review of my code.
I have commented my code, particularly in hard-to-understand areas.
I have made corresponding changes to the documentation.
I have checked for plagiarism and assure its authenticity.
I have read and followed the code of conduct for this repository. I understand that violation of this undertaking may have legal consequences.
I Agree

…tted for review

…tientJourney, Quiz, and Workflow - Implement CourseListingScreen for browsing and searching courses with category filtering. - Create HospitalMapScreen to visualize hospital departments and navigate to workflows. - Develop LessonReaderScreen for reading lessons with interactive elements and progress tracking. - Introduce PatientJourneyScreen to illustrate the patient journey through various hospital steps. - Add QuizScreen for knowledge checks with interactive question and answer functionality. - Implement WorkflowScreen to display department-specific workflows with step-by-step navigation.

…or better clarity

github-actions · 2026-07-11T16:48:10Z

Thank you @, for creating the PR and contributing to our UltimateHealth project 💗.
Our team will review the PR and will reach out to you soon! 😇
Make sure that you have marked all the tasks that you are done with ✅.
Thank you for your patience! 😀

github-actions · 2026-07-11T16:48:33Z

Automated Review Feedback

Provide actionable comments grouped by severity:

Important

Redundant Header Options in TabNavigation.tsx: For the Academy Tab.Screen, headerTransparent, headerStyle, and headerTitleStyle are specified, but headerShown: false is also set. When headerShown: false, these header styling options have no effect. Please remove these redundant options for clarity and maintainability.
Missing Tests: This pull request introduces a significant amount of new UI components and screens. There are no corresponding unit or integration tests included. Adding tests for critical components and screen navigation flows would greatly improve the robustness and maintainability of the new Academy module.

Suggestions

Stricter Typing for imageIcon: In src/components/Academy/CourseCard.tsx, the course.imageIcon prop is cast as any when used with MaterialCommunityIcons. It would be more robust to define a specific type for imageIcon (e.g., a union of valid icon names or a custom type that ensures it's a string compatible with MaterialCommunityIcons) to leverage TypeScript's type checking.
Inline Styling in HomeScreen.tsx: The new Academy featured card in src/screens/HomeScreen.tsx uses inline styling. For consistency and better maintainability, consider moving these styles into a StyleSheet.create object within the component or a shared stylesheet.
Clarify app.config.js Changes: The changes from /ap to /api in app.config.js appear to be corrections. If the /ap paths were actively used for deep links or API calls in a production environment, this change could be a breaking change for existing users or integrations. Please confirm that this change is intended as a correction and that the /ap paths are no longer in use or will be deprecated gracefully.

Maintainer Note:

Once the initial automated feedback has been addressed, maintainer @SB2318 will review the pull request for final evaluation.

…ter user experience

…rticle display

…creen and ProfileScreen for improved user experience

github-actions · 2026-07-14T04:29:37Z

Knip Dead Code Check

Knip ran in non-blocking mode for this PR. Review any findings before merge.

Check	Result	Exit code
Full analysis	PASS	`0`
Dependencies	PASS	`0`
Files	PASS	`0`
Exports	PASS	`0`

Full analysis

yarn run v1.22.22
$ knip
Configuration hints (3)
[App.tsx, …]                     knip.json  Remove, or move unused top-level entry to one of "workspaces"  
[src/**/*.{ts,tsx,js,jsx}, …]    knip.json  Remove, or move unused top-level project to one of "workspaces"
app.config.js                    knip.json  Remove redundant entry pattern                                 
Done in 1.22s.

Dependencies

yarn run v1.22.22
$ knip --dependencies
Configuration hints (3)
[App.tsx, …]                     knip.json  Remove, or move unused top-level entry to one of "workspaces"  
[src/**/*.{ts,tsx,js,jsx}, …]    knip.json  Remove, or move unused top-level project to one of "workspaces"
app.config.js                    knip.json  Remove redundant entry pattern                                 
Done in 1.09s.

Files

yarn run v1.22.22
$ knip --files
Unused files (28)
src/assets/css/article.ts                
src/assets/svg/EllipseSvg.jsx            
src/components/AddIcon.tsx               
src/components/AddSpecializationModal.tsx
src/components/AmplitudeWave.tsx         
src/components/APILoader.tsx             
src/components/BackToTopScrollView.tsx   
src/components/CustomStatusBar.tsx       
src/components/Dropdown.tsx              
src/components/DropDownComponent.tsx     
src/components/EmailVerificationModal.tsx
src/components/GlassInput.tsx            
src/components/HomeScreenHeader.tsx      
src/components/InactiveUserModal.tsx     
src/components/Input.tsx                 
src/components/Message.tsx               
src/components/MicWave.tsx               
src/components/NetworkCheck.tsx          
src/components/NoInternetLayout.tsx      
src/components/OverviewItem.tsx          
src/components/PodcastEmptyComponent.tsx 
src/components/PodcastPlayer.tsx         
src/components/SoundWave.tsx             
src/helper/MultiLanguageService.ts       
src/helper/notificationHandler.ts        
src/helper/PushNotificationService.ts    
src/helper/trackPlayerService.ts         
src/hooks/useGetReadHistory.ts           
Configuration hints (9)
[App.tsx, …]                     knip.json  Remove, or move unused top-level entry to one of "workspaces"  
[src/**/*.{ts,tsx,js,jsx}, …]    knip.json  Remove, or move unused top-level project to one of "workspaces"
**/*.config.{ts,js,mjs,cjs}      knip.json  Remove from ignore                                             
babel.config.js                  knip.json  Remove from ignore                                             
metro.config.js                  knip.json  Remove from ignore                                             
jest.config.js                   knip.json  Remove from ignore                                             
react-native.config.js           knip.json  Remove from ignore                                             
.eslintrc.js                     knip.json  Remove from ignore                                             
app.config.js                    knip.json  Remove redundant entry pattern                                 
Done in 1.19s.

Exports

yarn run v1.22.22
$ knip --exports
Configuration hints (3)
[App.tsx, …]                     knip.json  Remove, or move unused top-level entry to one of "workspaces"  
[src/**/*.{ts,tsx,js,jsx}, …]    knip.json  Remove, or move unused top-level project to one of "workspaces"
app.config.js                    knip.json  Remove redundant entry pattern                                 
Done in 1.13s.

Docs: docs/KNIP_GUIDE.md and docs/KNIP_CONTRIBUTOR_GUIDE.md

… elements.

SB2318 added 3 commits July 4, 2026 20:40

Update API path prefix in app configuration for consistency app submi…

2af3749

…tted for review

Refactor Academy screens to improve code structure and update icons f…

1b963cb

…or better clarity

github-actions Bot added gssoc in-progress level:frontend quality:clean type:feature labels Jul 11, 2026

SB2318 changed the title ~~Initial Implementation: Hospital Learning & Patient Management Academy (Mobile)~~ Feat: Hospital Learning & Patient Management Academy (Mobile) Jul 11, 2026

SB2318 changed the title ~~Feat: Hospital Learning & Patient Management Academy (Mobile)~~ feat: Hospital Learning & Patient Management Academy Initial (Mobile) Jul 11, 2026

SB2318 added 3 commits July 12, 2026 12:49

Add About and Settings screens, update navigation and profile for bet…

6740ba5

…ter user experience

Enhance ProfileScreen with tab navigation and FlatList for improved a…

0dddeef

…rticle display

Add ContentListScreen and integrate with navigation; update SettingsS…

197a3c6

…creen and ProfileScreen for improved user experience

feat: add RespectGiverScreen with animated components and interactive…

edbe055

… elements.

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

feat: Hospital Learning & Patient Management Academy Initial (Mobile) #2111

feat: Hospital Learning & Patient Management Academy Initial (Mobile) #2111
SB2318 wants to merge 7 commits into
mainfrom
feature/ai-persona-chat

SB2318 commented Jul 11, 2026

Uh oh!

github-actions Bot commented Jul 11, 2026

Uh oh!

github-actions Bot commented Jul 11, 2026

Uh oh!

github-actions Bot commented Jul 14, 2026 •

edited

Loading

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

1 participant

Conversation

SB2318 commented Jul 11, 2026

PR Description

Changes Included

Type of Change

Select your work-area

Related Issue

Add your Work Example

Fixes (mention the issue number which this fixes)

Checklist

Undertaking

Uh oh!

github-actions Bot commented Jul 11, 2026

Uh oh!

github-actions Bot commented Jul 11, 2026

Automated Review Feedback

Important

Suggestions

Uh oh!

github-actions Bot commented Jul 14, 2026 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Knip Dead Code Check

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

1 participant

github-actions Bot commented Jul 14, 2026 •

edited

Loading