Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Patient privacy feature #6021

Closed

Conversation

JahnabDutta
Copy link
Contributor

@JahnabDutta JahnabDutta commented Aug 7, 2023

WHAT

🤖 Generated by Copilot at 0657245

This pull request adds features and improvements to the camera and bed management in the facility dashboard. It replaces the CameraConfigure component with the LiveFeed component to show the camera feed and the presets. It also adds a new component, CameraBoundaryConfigure, to handle the boundary preset for a bed. It adds new fields and types to the bed and camera models and interfaces. It also adds a feature to toggle the privacy status of a patient in the bed using a new action and a new API route.

Proposed Changes

@coronasafe/care-fe-code-reviewers @coronasafe/code-reviewers

Merge Checklist

  • Add specs that demonstrate bug / test a new feature.
  • Update product documentation.
  • Ensure that UI text is kept in I18n files.
  • Prep screenshot or demo video for changelog entry, and attach it to issue.
  • Request for Peer Reviews
  • Completion of QA

HOW

🤖 Generated by Copilot at 0657245

  • Add types, components, actions, and routes to implement the camera presets and boundary preset feature for the ONVIF camera (link, link, link, link, link, link, link, link, link, link, link, link)
  • Modify the ONVIFCamera component to fetch, create, update, and delete the presets and the boundary preset for a bed using the LiveFeed component (link, link, link, link, link, link, link)
  • Remove the CameraConfigure component from the ONVIFCamera component and simplify its layout and logic to only show the input field and buttons to add or cancel a preset (link, link)
  • Add the CameraBoundaryConfigure component to render the buttons and labels to add, update, or delete the boundary preset for a bed (link)
  • Add the privacy field to the CurrentBed interface and the togglePatientPrivacy action to implement the privacy toggle feature for the patient in the bed (link, link)
  • Modify the PatientInfoCard component to fetch, display, and update the privacy status of the patient using the SwitchTabs component (link, link, link, link, link, link, link)
  • Add the className prop and the classNames function to the SwitchTabs component to allow custom class names for the component (link, link)
  • Add the meta field to the BedModel interface to store the metadata of the bed, such as the boundary preset (link)
  • Add the toAddPreset prop to the LiveFeed component to indicate whether to show the CameraConfigure component or not (link)
  • Fix a typo in the class name of the div element that contains the patient name in the PatientInfoCard component (link)
  • Add a blank div element to the PatientInfoCard component to create a gap between the patient name and the switch tabs for the privacy toggle feature (link)
  • Add a blank line to the ONVIFCamera component for readability (link)

JahnabDutta and others added 30 commits May 26, 2023 13:07
Signed-off-by: JahnabDutta <[email protected]>
Signed-off-by: JahnabDutta <[email protected]>
Signed-off-by: JahnabDutta <[email protected]>
@JahnabDutta JahnabDutta marked this pull request as ready for review August 29, 2023 16:19
@github-actions github-actions bot added merge conflict pull requests with merge conflict and removed merge conflict pull requests with merge conflict labels Aug 30, 2023
@github-actions
Copy link

👋 Hi, @JahnabDutta,
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.

Signed-off-by: JahnabDutta <[email protected]>
Signed-off-by: JahnabDutta <[email protected]>
Signed-off-by: JahnabDutta <[email protected]>
@github-actions github-actions bot removed the merge conflict pull requests with merge conflict label Aug 30, 2023
@github-actions github-actions bot added the merge conflict pull requests with merge conflict label Sep 7, 2023
@github-actions
Copy link

github-actions bot commented Sep 7, 2023

👋 Hi, @JahnabDutta,
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.

@nihal467
Copy link
Member

a single pr is already raised for this issue : #6223

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
merge conflict pull requests with merge conflict needs testing
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants