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

Feat: Added testimonials #99

Merged
merged 5 commits into from
Jul 2, 2024

Conversation

nagalakshmi08
Copy link
Contributor

Closes #98
Hi, I have added the testimonials section which matches with ui and theme of website and are also mobile responsive.

Screenshot 2024-06-13 110111

Mobile view:
Screenshot 2024-06-13 110042

Recording.2024-06-13.105924.mp4

Please check it and let me know if any changes to be made. Thank you!

Copy link

vercel bot commented Jun 13, 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 Jun 23, 2024 5:53am

@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.

Do not modify the 'package.json' file. Update to the latest code, and then I will merge.

@nagalakshmi08
Copy link
Contributor Author

Do not modify the 'package.json' file. Update to the latest code, and then I will merge.

Please check it now.

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.

You should improve the design of (responsive) arrow buttons.
image

@nagalakshmi08
Copy link
Contributor Author

You should improve the design of (responsive) arrow buttons. image

I have increased the padding , now it looks good in all types of screens.

@BamaCharanChhandogi
Copy link
Owner

Alright. This issue is showing in the console, please Fix it. Then I will merge.

image

@BamaCharanChhandogi
Copy link
Owner

why some images are not showing?

image

@nagalakshmi08
Copy link
Contributor Author

why some images are not showing?

image

Can you please refresh and check it once, all images are being displayed to me.

@BamaCharanChhandogi
Copy link
Owner

I have refreshed. It's still not showing images.

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 aa0d746 into BamaCharanChhandogi:main Jul 2, 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.

Feat: Adding a testimonials section.
2 participants