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

Make Treating Doctor Field Optional for Domiciliary Care Patients #8304 #8360

Closed
wants to merge 0 commits into from

Conversation

JOSHIK27
Copy link
Contributor

@JOSHIK27 JOSHIK27 commented Aug 19, 2024

Required Backends

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

@JOSHIK27 JOSHIK27 requested a review from a team as a code owner August 19, 2024 20:12
Copy link

netlify bot commented Aug 19, 2024

Deploy Preview for care-egov-staging ready!

Name Link
🔨 Latest commit a65681e
🔍 Latest deploy log https://app.netlify.com/sites/care-egov-staging/deploys/66c6e16a72bb62000876210c
😎 Deploy Preview https://deploy-preview-8360--care-egov-staging.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site configuration.

@rithviknishad
Copy link
Member

@JOSHIK27 Could you also update the PR's title to be something more meaningful 😄

@JOSHIK27 JOSHIK27 changed the title Fixed #8304 Make Treating Doctor Field Optional for Domiciliary Care Patients #8304 Aug 20, 2024
@JOSHIK27
Copy link
Contributor Author

@JOSHIK27 Could you also update the PR's title to be something more meaningful 😄

done @rithviknishad

Copy link

vercel bot commented Aug 20, 2024

@JOSHIK27 is attempting to deploy a commit to the Open Healthcare Network Team on Vercel.

A member of the Team first needs to authorize it.

@nihal467
Copy link
Member

@JOSHIK27
image

its throwing a 500 error in the frontend

CC: @rithviknishad

@rithviknishad
Copy link
Member

@nihal467 backend changes required: ohcnetwork/care#2382

Could you test with that?

@rithviknishad
Copy link
Member

@JOSHIK27 would you be making a new PR for this?

@JOSHIK27
Copy link
Contributor Author

@JOSHIK27 would you be making a new PR for this?

Yes. Definitely

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Treating Doctor field should not be mandatory for Domiciliary Care patients
3 participants