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

FIXED Issue #5 : Added a Contact Us Page #33

Merged

Conversation

AkshitGarg24
Copy link
Contributor

Fixes #5

A dedicated "Contact Us" page has been created. First, a "Contact Us" button has been added to the navigation bar. When this button is clicked, it opens a "Contact Us" page that provides details such as email, contact number, business hours, and the address of the organization. Additionally, a contact form has been included to allow users to get in touch directly.

Request you to please label the PR before merging it. Please label the level of the PR according to the design made.

Thank You! Happy Coding!

ContactUs.mp4

Copy link

vercel bot commented May 28, 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 May 30, 2024 11:33am

@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! 😊

@BamaCharanChhandogi
Copy link
Owner

Hi @AkshitGarg24,
can you change the design part, I think the Contact Us page is not looking well.

@AkshitGarg24
Copy link
Contributor Author

@BamaCharanChhandogi Could you please provide more details about how you want this page design to look? Mention a few points, and I'll make the required changes. Alternatively, if you prefer, I can handle the redesign myself.

@BamaCharanChhandogi
Copy link
Owner

BamaCharanChhandogi commented May 28, 2024

@BamaCharanChhandogi Could you please provide more details about how you want this page design to look? Mention a few points, and I'll make the required changes. Alternatively, if you prefer, I can handle the redesign myself.

Sure. can you apply this type of simple design?

image

@AkshitGarg24
Copy link
Contributor Author

@BamaCharanChhandogi Could you please provide more details about how you want this page design to look? Mention a few points, and I'll make the required changes. Alternatively, if you prefer, I can handle the redesign myself.

Sure. can you apply this type of simple design?

image

Yaa Sure! I will do that and revert back to you.

Thank You!

…t-Us-Page

Merged updates from origin/5/Contact-us-Page to synchronize changes.
@AkshitGarg24
Copy link
Contributor Author

@BamaCharanChhandogi The required changes have been made. Please review and let me know if anything else is needed.

ContactUs.mp4

@BamaCharanChhandogi
Copy link
Owner

BamaCharanChhandogi commented May 29, 2024

@BamaCharanChhandogi The required changes have been made. Please review and let me know if anything else is needed.

ContactUs.mp4

Thanks for the changes. However, our project used tailwind CSS. So please write it in tailwind instead of using normal CSS.
Don't worry I will increase the level.

Or one thing why you change package JSON file.

@AkshitGarg24
Copy link
Contributor Author

The page has now been implemented using Tailwind CSS. Please review the changes and let me know if any further adjustments are needed.

I did not make any changes to the package.json file. It might get modified automatically when implementing the Contact-Us page.

@BamaCharanChhandogi
Copy link
Owner

The page has now been implemented using Tailwind CSS. Please review the changes and let me know if any further adjustments are needed.

I did not make any changes to the package.json file. It might get modified automatically when implementing the Contact-Us page.

Alright. but You have to remove the changes in the package JSON file. neither I can't merge.

@AkshitGarg24
Copy link
Contributor Author

The only thing I can do is copy the contents of the package.json file from the main project and then paste it here and then making a commit. Will that be okay, @BamaCharanChhandogi ?

@BamaCharanChhandogi
Copy link
Owner

The only thing I can do is copy the contents of the package.json file from the main project and then paste it here and then making a commit. Will that be okay, @BamaCharanChhandogi ?

Yeah, I think you can do it, let's see.

@AkshitGarg24
Copy link
Contributor Author

@BamaCharanChhandogi I have removed changes from package-lock.json file. You can check 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.

Looks good, Thanks.

@BamaCharanChhandogi BamaCharanChhandogi merged commit c84dac0 into BamaCharanChhandogi:main May 30, 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.

Adding a "Contact Us" Page
2 participants