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

Bug: contact us button is broken in footer #1131

Closed
3 tasks done
MahendraDani opened this issue Oct 12, 2023 · 12 comments · Fixed by #1140
Closed
3 tasks done

Bug: contact us button is broken in footer #1131

MahendraDani opened this issue Oct 12, 2023 · 12 comments · Fixed by #1140
Assignees
Labels
🐞 bug This is a patch, generally fixing some bugs. hacktoberfest This issue is a part of hacktoberfest 2023. status: wip 👩🏻‍💻 This issue/PR is actively being worked on.

Comments

@MahendraDani
Copy link
Contributor

Description

  • When the contact us button in footer is clicked the website becomes blank. The button neither redirects to any page nor shows any contact form. It seems like the website hangs out and user needs to refresh to see the website live again.

  • Steps to Reproduce

  1. Go to https://milanhub.org/
  2. Scroll down to footer of the website
  3. Click on contact us button
  • Screen recording
0f77e36f-5572-42f9-bba7-b5dfec384f2b.mp4

Proposed Solution

Solution 1

We can show Work in progress as shown in https://milanhub.org/shop

Solution 2

We can develop a contact us page and create a contact form which will open when contact us button is clicked.

Screenshots

Screen shot for proposed solution 1
milan_contact

Browser 🥦

Google Chrome

Checklist ✅

@MahendraDani MahendraDani added hacktoberfest This issue is a part of hacktoberfest 2023. status: todo ⏳ This issue is yet to be seen by the maintainer of the project. 🐞 bug This is a patch, generally fixing some bugs. labels Oct 12, 2023
@github-actions
Copy link

Hello @MahendraDani, thank you for raising the issue.

Currently, the issue is marked as https://github.com/MilanCommunity/Milan/labels/status%3A%20todo%20%E2%8F%B3 so please wait until the maintainers/owner review it and provide you with feedback/suggestions to proceed further.

Feel free to reach out to Tamal on Twitter, or drop a mail at tamalcodes@gmail.com if you think that this issue is of critical priority.

Give us a ⭐ to show some support
Happy OpenSource 🚀

@VishalMyla
Copy link

please assign this issue to me

@MahendraDani
Copy link
Contributor Author

@VishalMyla
I will be working on this issue.
Please checkout other open issues.
Thanks!

@tamalCodes
Copy link
Member

tamalCodes commented Oct 13, 2023

This is being worked on in #1082 , if they are inactive for a few more days I'll assign it to you. Until then please stay on hold. Do not make PR for the same for now.

@tamalCodes tamalCodes added status: on hold 🚦 This issue/PR is on hold/blocked for now as other issues/PRs needs to be merged before this. and removed status: todo ⏳ This issue is yet to be seen by the maintainer of the project. labels Oct 13, 2023
@MahendraDani
Copy link
Contributor Author

This is being worked on in #1082 , if they are inactive for a few more days I'll assign it to you. Until then please stay on hold. Do not make PR for the same for now.

@tamalCodes alright

@MahendraDani
Copy link
Contributor Author

@tamalCodes Should I start working on this issue now!

@tamalCodes
Copy link
Member

Sure go ahead, please take a look at this issue to have a more understanding of what we desire to have. In case of more doubts please let me know !

📑 Checklist in case you missed something

Please maintain structure, use conventional pull request titles, and avoid spam to avoid closure or bans.

@tamalCodes tamalCodes added status: wip 👩🏻‍💻 This issue/PR is actively being worked on. and removed status: on hold 🚦 This issue/PR is on hold/blocked for now as other issues/PRs needs to be merged before this. labels Oct 16, 2023
@MahendraDani
Copy link
Contributor Author

@tamalCodes Thanks.
Would you please tell me which input fields should be there fir users in the form when the modal opens after the button is clicked.
For example -
name,email,query , etc?

@tamalCodes
Copy link
Member

I am planning to use the Report schema for the same, so the details can be the same as what we have in the schema, and API call would also be needed, one might be already present in the Frontend code.

In the backend, we have the Report route which can be used.

@MahendraDani
Copy link
Contributor Author

MahendraDani commented Oct 16, 2023

@tamalCodes
So I need to develop the UI of contact us form and then connect it to report route in the backend right?

@tamalCodes
Copy link
Member

@tamalCodes So I need to develop the UI of contact us form and then connect it to report route in the backend right?

yes the submit button will just send over that data. You can use this already coded function too for the same.

@MahendraDani
Copy link
Contributor Author

@tamalCodes So I need to develop the UI of contact us form and then connect it to report route in the backend right?

yes the submit button will just send over that data. You can use this already coded function too for the same.

Alright!
I will try to complete this asap.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
🐞 bug This is a patch, generally fixing some bugs. hacktoberfest This issue is a part of hacktoberfest 2023. status: wip 👩🏻‍💻 This issue/PR is actively being worked on.
Projects
Status: Merged/Discarded
Development

Successfully merging a pull request may close this issue.

4 participants
@tamalCodes @VishalMyla @MahendraDani and others