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

UI-Enhancement #113

Merged

Conversation

Taranpreet10451
Copy link
Contributor

Completed the task.

Sir I will add the description regarding the parameters mentioned on the Predictions page for predicting whether the person is diabetic or not. The details of the parameters will include why they are use for prediction, in place of the GIF image.

Screenshot 2024-06-13 231238

Completed the task.
Copy link

vercel bot commented Jun 21, 2024

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

Name Status Preview Comments Updated (UTC)
diabetes-prediction ✅ Ready (Inspect) Visit Preview 💬 Add feedback Jul 3, 2024 1:56pm

@BamaCharanChhandogi
Copy link
Owner

Thank you for submitting your pull request! 🙌 We'll review it as soon as possible. In the meantime, please ensure that your changes align with our CONTRIBUTING.md. If there are any specific instructions or feedback regarding your PR, we'll provide them here. Thanks again for your contribution! 😊

Copy link
Owner

@BamaCharanChhandogi BamaCharanChhandogi left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

why did you change the `packege.json file? And make it responsive.

@Taranpreet10451
Copy link
Contributor Author

@BamaCharanChhandogi Sir please review it I have made the changes.

@BamaCharanChhandogi
Copy link
Owner

BamaCharanChhandogi commented Jul 3, 2024

please fix the merge conflict.

@Taranpreet10451
Copy link
Contributor Author

Taranpreet10451 commented Jul 3, 2024

@BamaCharanChhandogi
Sir, please review it once again. Please label it also.

Copy link
Owner

@BamaCharanChhandogi BamaCharanChhandogi left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM, thanks.

@BamaCharanChhandogi BamaCharanChhandogi merged commit e1d657f into BamaCharanChhandogi:main Jul 6, 2024
2 checks passed
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.

2 participants