-
Notifications
You must be signed in to change notification settings - Fork 480
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
fix: Show validation error message on facility cover image upload (#8625) #8639
fix: Show validation error message on facility cover image upload (#8625) #8639
Conversation
✅ Deploy Preview for care-ohc ready!
To edit notification comments on pull requests, go to your Netlify site configuration. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Can we instead show the error from the backend as is as mentioned in issue comment instead of duplicating the validations and checks from client too?
sure |
here is the demo of the video Recording.2024-09-26.195959.1.mp4 |
@rithviknishad how to fix these errors |
LGTM |
@syedfardeenjeelani Your efforts have helped advance digital healthcare and TeleICU systems. 🚀 Thank you for taking the time out to make CARE better. We hope you continue to innovate and contribute; your impact is immense! 🙌 |
Proposed Changes
Video Demo:
https://github.com/user-attachments/assets/f0c8d680-8885-4856-a2ee-77fbebd27964
@ohcnetwork/care-fe-code-reviewers
@nihal467
Merge Checklist