Open shivankacker opened 2 months ago
netlify[bot]
commented
2 months ago | Name | Link |
|---|---|
| Latest commit | |
| Latest deploy log | https://app.netlify.com/sites/care-ohc/deploys/672bcc65dff07e3bed14af2a |
| Deploy Preview | https://deploy-preview-8469--care-ohc.netlify.app |
| Preview on mobile | Toggle QR Code...Use your smartphone camera to open QR code link. |
To edit notification comments on pull requests, go to your Netlify site configuration.
github-actions[bot]
commented
2 months ago :wave: Hi, @shivankacker, Conflicts have been detected against the base branch. Please rebase your branch against the base branch.
This message is automatically generated by prince-chrismc/label-merge-conflicts-action so don't hesitate to report issues/improvements there.
cypress[bot]
commented
1 month ago 
Run #3834
Passed #3834
2771ec63fc: Migrated Scribe to a CARE App + Context| Project |
CARE
|
| Branch Review |
scribe-context
|
| Run status |
|
| Run duration | 04m 35s |
| Commit |
|
| Committer | Shivank Kacker |
| View all properties for this run ↗︎ | |
| Test results | |
|---|---|
Failures
|
0
|
Flaky
|
0
|
Pending
|
0
|
Skipped
|
0
|
|
Passing
|
125
|
| View all changes introduced in this branch ↗︎ | |
github-actions[bot]
commented
1 month ago :wave: Hi, @shivankacker, Conflicts have been detected against the base branch. Please rebase your branch against the base branch.
This message is automatically generated by prince-chrismc/label-merge-conflicts-action so don't hesitate to report issues/improvements there.
bodhish
commented
1 month ago @shivankacker please close pending PR's before opening others.
shivankacker
commented
1 month ago @bodhish working on this currently
cloudflare-workers-and-pages[bot]
commented
3 weeks ago 
Cloudflare Pages| Latest commit: |
2771ec6
|
| Status: | ✅ Deploy successful! |
| Preview URL: | https://16504526.care-fe.pages.dev |
| Branch Preview URL: | https://scribe-context.care-fe.pages.dev |
github-actions[bot]
commented
2 weeks ago :wave: Hi, @shivankacker, Conflicts have been detected against the base branch. Please rebase your branch against the base branch.
This message is automatically generated by prince-chrismc/label-merge-conflicts-action so don't hesitate to report issues/improvements there.
github-actions[bot]
commented
1 week ago :wave: Hi, @shivankacker, Conflicts have been detected against the base branch. Please rebase your branch against the base branch.
This message is automatically generated by prince-chrismc/label-merge-conflicts-action so don't hesitate to report issues/improvements there.
github-actions[bot]
commented
1 week ago :wave: Hi, @shivankacker, Conflicts have been detected against the base branch. Please rebase your branch against the base branch.
This message is automatically generated by prince-chrismc/label-merge-conflicts-action so don't hesitate to report issues/improvements there.
coderabbitai[bot]
commented
3 days ago The pull request introduces multiple enhancements across various components in the application. Key changes include the addition of new hooks and components for improved keyboard shortcut handling, audio recording management, and value observation in form fields. The Scribe component has been completely removed, along with associated API routes, indicating a significant shift in how audio and form data are managed. Other modifications involve updates to existing components for better user experience and responsiveness, particularly in relation to input handling and data synchronization.
| File Path | Change Summary |
|---|---|
src/CAREUI/interactive/KeyboardShortcut.tsx |
Added KeyboardShortcutKey function to render keyboard shortcuts with optional short key display. |
src/Routers/AppRouter.tsx |
Added data-cui-page attribute to the main <div> element. |
src/Utils/useValueInjectionObserver.tsx |
Introduced a new hook to observe changes to a specified DOM element's attribute. |
src/Utils/useVoiceRecorder.ts |
Implemented a custom hook for audio recording management with state handling and permissions. |
src/components/Common/DateInputV2.tsx |
Integrated useValueInjectionObserver for improved date input responsiveness. |
src/components/Common/prescription-builder/PrescriptionDropdown.tsx |
Added key prop to dropdown options for rendering performance. |
src/components/Files/AudioCaptureDialog.tsx |
Refactored to use useVoiceRecorder hook for audio recording management. |
src/components/Form/FormFields/Autocomplete.tsx |
Enhanced value management with useValueInjectionObserver and improved accessibility. |
src/components/Form/FormFields/AutocompleteMultiselect.tsx |
Updated to use useValueInjectionObserver for dropdown value observation. |
src/components/Form/FormFields/RadioFormField.tsx |
Changed unselect radio input's value from undefined to "__NULL__". |
src/components/Form/FormFields/RangeAutocompleteFormField.tsx |
Made Threshold interface public. |
src/components/Form/FormFields/TextFormField.tsx |
Added optional thresholds property for enhanced validation. |
src/components/Form/SelectMenuV2.tsx |
Enhanced responsiveness with useValueInjectionObserver. |
src/components/Patient/DailyRounds.tsx |
Significant refactor, replacing several components and updating form handling logic. |
src/components/Scribe/Scribe.tsx |
Complete removal of the Scribe component. |
src/components/Scribe/formDetails.ts |
Removed DAILY_ROUND_FORM_SCRIBE_DATA structure and SCRIBE_FORMS export. |
src/components/Symptoms/SymptomsBuilder.tsx |
Updated to improve user interaction and added tracking attributes. |
src/pluginTypes.ts |
Added new type ScribeComponentType and included it in SupportedPluginComponents. |
tailwind.config.js |
Simplified content property format in Tailwind CSS configuration. |
cypress/pageobject/Patient/PatientLogupdate.ts |
Updated input handling methods for pulse, respiratory rate, and SpO2. |
src/components/Common/BloodPressureFormField.tsx |
Adjusted separator styling between input fields. |
src/components/Common/TemperatureFormField.tsx |
Enhanced input handling and conversion logic for temperature. |
src/Utils/useSegmentedRecorder.ts |
Added microphoneAccess state to manage microphone permission status. |
cypress/e2e/assets_spec/AssetHomepage.cy.ts |
Reordered import statements for better organization. |
cypress/e2e/assets_spec/AssetsCreation.cy.ts |
Reordered import statements for clarity. |
cypress/e2e/assets_spec/AssetsManage.cy.ts |
Reordered import statements without functional changes. |
cypress/e2e/facility_spec/FacilityCreation.cy.ts |
Reordered import statements for better organization. |
cypress/e2e/facility_spec/FacilityHomepage.cy.ts |
Updated imports and added pagination-related tests. |
cypress/e2e/facility_spec/FacilityInventory.cy.ts |
Reordered import statements for better organization. |
cypress/e2e/facility_spec/FacilityManage.cy.ts |
Reordered import statements for clarity. |
cypress/e2e/patient_spec/PatientBedManagement.cy.ts |
Activated import for PatientConsultationPage for enhanced test capabilities. |
cypress/e2e/patient_spec/PatientConsultationCreation.cy.ts |
Reorganized import statements for better readability. |
cypress/e2e/patient_spec/PatientDoctorConnect.cy.ts |
Introduced a new test suite for patient-doctor connection functionality. |
cypress/e2e/patient_spec/PatientFileUpload.ts |
Added tests for audio and file uploads in patient management. |
cypress/e2e/patient_spec/PatientLogUpdate.cy.ts |
Re-added import for PatientLogupdate class. |
cypress/e2e/patient_spec/PatientPrescription.cy.ts |
Re-added import for PatientPrescription class. |
cypress/e2e/patient_spec/PatientRegistration.cy.ts |
Enhanced import organization and patient transfer functionality. |
cypress/e2e/resource_spec/ResourcesHomepage.cy.ts |
Reordered import statements for better organization. |
cypress/e2e/users_spec/UsersCreation.cy.ts |
Reorganized import statements and added utility function. |
cypress/e2e/users_spec/UsersManage.cy.ts |
Added import for UserPage to enhance user management functionality. |
public/locale/en.json |
Added new key-value pairs for enhanced localization support. |
src/Utils/request/api.tsx |
Removed several API routes related to "Scribe" functionalities. |
sequenceDiagram
participant User
participant App
participant Scribe
User->>App: Interact with form
App->>Scribe: Manage audio and form data
Scribe-->>App: Provide feedback
App-->>User: Display updated form| Objective | Addressed | Explanation |
|---|---|---|
| Make Scribe Aware of already present form data (#8134) | ❌ | Scribe component has been completely removed. |
| Enhancement to the CARE Scribe (#8285) | ❌ | Scribe functionalities have been removed entirely. |
| Replace unnecessary range auto complete form fields in Daily Rounds with integer fields (#8978) | ✅ | Form fields replaced as per the issue requirements. |
| Scribe is malfunctioning in iphone (#8227) | ❌ | Removal of Scribe component does not address this. |
KeyboardShortcutKey function in the main PR.tested, good to merge
🐇 In the land of code where bunnies play,
New features hop in, brightening the day.
With shortcuts and voices, our tasks are a breeze,
Removing the Scribe, we do it with ease.
So here's to the changes, both big and small,
In our code garden, we cherish them all! 🌼
Thank you for using CodeRabbit. We offer it for free to the OSS community and would appreciate your support in helping us grow. If you find it useful, would you consider giving us a shout-out on your favorite social media?
github-actions[bot]
commented
2 days ago :wave: Hi, @shivankacker, Conflicts have been detected against the base branch. Please rebase your branch against the base branch.
This message is automatically generated by prince-chrismc/label-merge-conflicts-action so don't hesitate to report issues/improvements there.
shivankacker
commented
2 days ago Will merge conflicts once PR is tested and approved
nihal467
commented
2 days ago @shivankacker can you clear the merge conflict, to get it ready for review
shivankacker
commented
2 days ago @nihal467 resolved merge conflicts, deploy preview failing due to cache issue, nothing else.
github-actions[bot]
commented
2 hours ago :wave: Hi, @shivankacker, Conflicts have been detected against the base branch. Please rebase your branch against the base branch.
This message is automatically generated by prince-chrismc/label-merge-conflicts-action so don't hesitate to report issues/improvements there.
Proposed Changes
Moves Scribe to https://github.com/ohcnetwork/care_scribe_fe
https://github.com/user-attachments/assets/8bbfe6d7-c8be-4a14-bdb5-30ccad3dc4e7
Requires care_scribe backend plugin pointing to https://github.com/ohcnetwork/care_scribe/pull/12
Requires the care_scribe_fe frontend appp. Add that by updating your env to include
REACT_ENABLED_APPS="ohcnetwork/care_livekit_fe,ohcnetwork/care_scribe_fe"NOTE: Cannot update non arbitrary fields like symptoms, diagnoses for now.
@ohcnetwork/care-fe-code-reviewers
Merge Checklist
Summary by CodeRabbit
Release Notes
New Features
Bug Fixes
Chores