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

Disable submit button in round type normal if no value selected #6793

Merged
merged 3 commits into from
Dec 13, 2023

Conversation

Pranshu1902
Copy link
Contributor

@Pranshu1902 Pranshu1902 commented Dec 4, 2023

WHAT

🤖[deprecated] Generated by Copilot at f3c1ec2

Enable different rounds types and form validation for daily rounds. Refactor Submit component in DailyRounds.tsx.

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

🤖[deprecated] Generated by Copilot at f3c1ec2

  • Add a disabled state to control the submit button for daily rounds form (link)
  • Enable the submit button when the rounds type is not "NORMAL" (link)
  • Pass the disabled state and the rounds_type value to the Submit component as props (link)
  • Disable the submit button in the Submit component when the rounds type is "NORMAL" and the form is not filled (link)
  • Fix the indentation of the code in DailyRounds.tsx (link)

@Pranshu1902 Pranshu1902 requested a review from a team as a code owner December 4, 2023 16:20
Copy link

vercel bot commented Dec 4, 2023

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Comments Updated (UTC)
care-storybook ✅ Ready (Inspect) Visit Preview 💬 Add feedback Dec 12, 2023 5:25pm

Copy link

netlify bot commented Dec 4, 2023

Deploy Preview for care-egov-staging ready!

Name Link
🔨 Latest commit 83255f9
🔍 Latest deploy log https://app.netlify.com/sites/care-egov-staging/deploys/657897499d437500081a2d76
😎 Deploy Preview https://deploy-preview-6793--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.

@nihal467
Copy link
Member

nihal467 commented Dec 6, 2023

@Pranshu1902
image

the above patient have previous record, i haven't copied any record, i didn't filled any fields from physical examination info till rhythm description, still the save button is active,

it is expected to be disabled if no data was entered in any of the fields

@Pranshu1902
Copy link
Contributor Author

@nihal467 Bug resolved
image

@nihal467
Copy link
Member

nihal467 commented Dec 9, 2023

image

@Pranshu1902 @rithviknishad marking it for discussion as now in critical care, the above fields a mandatory thing to fill any of the field, need to get a final confirmation from @gigincg

@rithviknishad
Copy link
Member

rithviknishad commented Dec 12, 2023

@nihal467 @Pranshu1902 as discussed,

  • Critical care is an exception, you can skip this validation in Daily Rounds form.
  • However, in Critical Care editor, this similar validation is to be present. Any one fields must be changed (apart from the "Basic Editor").

cc: @aparnacoronasafe

@Pranshu1902
Copy link
Contributor Author

@nihal467 the form now isn't disabled for CRITICAL CARE

@nihal467
Copy link
Member

LGTM

@khavinshankar khavinshankar merged commit 4f39ce2 into ohcnetwork:develop Dec 13, 2023
27 of 36 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Disable Submit button in normal log update
4 participants