Skip to content

BMI: Improve Input Validation and Error Handling#46

Merged
venisha-kalola merged 1 commit into
commitra:mainfrom
Pawankalyan2023:main
Oct 15, 2025
Merged

BMI: Improve Input Validation and Error Handling#46
venisha-kalola merged 1 commit into
commitra:mainfrom
Pawankalyan2023:main

Conversation

@Pawankalyan2023
Copy link
Copy Markdown
Contributor

@Pawankalyan2023 Pawankalyan2023 commented Oct 14, 2025

Summary

PR for Issue no : #23

I have changed the validation part and also included the invalid border so as to indicate the issue clearly while entering data , also changed the styles on the UI part

Checklist

  • Linked issue (if any)
  • Ran locally without console errors
  • Focused scope (kept PR small)
  • Updated docs or TODO comments if needed

Screenshots

When both the condition fails :
image

when one of the condition fails :
image

if any one of the condition or both the condition fails it will make the border red , and the calculation won't happen when there is a invalid process

@vercel
Copy link
Copy Markdown

vercel Bot commented Oct 14, 2025

@Pawankalyan2023 is attempting to deploy a commit to the hrishikesh-dalal's projects Team on Vercel.

A member of the Team first needs to authorize it.

@venisha-kalola
Copy link
Copy Markdown
Contributor

venisha-kalola commented Oct 15, 2025

add the assigned issue number in the PR request, #(no.) see the original issue number
complete the checklist , add screenshots of other invalid inputs too

@Pawankalyan2023
Copy link
Copy Markdown
Contributor Author

add the assigned issue number in the PR request, #(no.) see the original issue number complete the checklist , add screenshots of other invalid inputs too

I have updated the PR , please do check

@venisha-kalola venisha-kalola merged commit 9f7a17b into commitra:main Oct 15, 2025
1 of 9 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants