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

Contact Us form added with Formsubmit for receiving form data #173

Closed
wants to merge 3 commits into from
Closed

Contact Us form added with Formsubmit for receiving form data #173

wants to merge 3 commits into from

Conversation

AnupK1234
Copy link

@AnupK1234 AnupK1234 commented Dec 13, 2023

Created a responsive Contact Us form and integrated it with Formsubmit which will send the form data to a user's email id.

Closes #57

[Feature] Contact us section

Screenshot

Screenshot 2023-12-14 013710

Check List (Check all the applicable boxes)

  • My code follows the code style of this project.
  • My change requires changes to the documentation.
  • I have updated the documentation accordingly.
  • All new and existing tests passed.

Copy link

netlify bot commented Dec 13, 2023

Deploy Preview for ar-webstore failed.

Name Link
🔨 Latest commit 9c10c5b
🔍 Latest deploy log https://app.netlify.com/sites/ar-webstore/deploys/657f492968601b0008be92a8

@ShwetKhatri2001
Copy link
Owner

Good work, but there is a scope of improvement.

First, it's sending me an email to activate the form, but when I open it, it's saying 'Not a valid link' ( Confirmation token not found.). Test it at your end using your mail id.

Second, try to improve the UI and use current theme colours like white, black or gray.
You may have input fields as present in the Feedback form and change the submit button as well.

@AnupK1234
Copy link
Author

Update: UI of Contact form changed, form submission working perfect.
Screenshot 2023-12-17 235526

@ShwetKhatri2001
Copy link
Owner

ShwetKhatri2001 commented Dec 17, 2023

Hey ! I'd like to merge this PR, but there was a .env file and I thought it was unnecessary. So I removed it, but it's causing an issue in the deployment. Are you using that file ? Why is that required ?

@ShwetKhatri2001 ShwetKhatri2001 mentioned this pull request Dec 19, 2023
5 tasks
@ShwetKhatri2001
Copy link
Owner

@AnupK1234 Will you be able to look at this within next few days ?

@AnupK1234
Copy link
Author

@AnupK1234 Will you be able to look at this within next few days ?

Ok Assign me this.

@ShwetKhatri2001
Copy link
Owner

Just fix it on this PR or raise a new one if you want to. You're already assigned the issue.

@ShwetKhatri2001
Copy link
Owner

@AnupK1234 , please try to fix it within the next 3 days.

@ShwetKhatri2001
Copy link
Owner

Closing this because of inactivity.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[Feature] Contact us section
2 participants