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

[Feature] Added Contact Form #92

Closed
wants to merge 7 commits into from

Conversation

KIET7UKE
Copy link

@KIET7UKE KIET7UKE commented Oct 9, 2022

Hey, @SaptarshiSarkar12 I have added the contact form to your website. Kindly verifies it.

  1. Light Mode
    Screenshot (1103)
  2. Dark Mode
    Screenshot (1104)

@SaptarshiSarkar12 SaptarshiSarkar12 temporarily deployed to production October 9, 2022 06:10 Inactive
@KIET7UKE
Copy link
Author

KIET7UKE commented Oct 9, 2022

Any Issues @SaptarshiSarkar12

@SaptarshiSarkar12 SaptarshiSarkar12 temporarily deployed to production October 9, 2022 06:14 Inactive
@SaptarshiSarkar12
Copy link
Owner

SaptarshiSarkar12 commented Oct 9, 2022

Any Issues @SaptarshiSarkar12

I don't know why I can't deploy previews of PRs. Can you see the workflow file and tell me if there is any problem?

@SaptarshiSarkar12 SaptarshiSarkar12 temporarily deployed to production October 9, 2022 06:22 Inactive
@SaptarshiSarkar12
Copy link
Owner

Any Issues @SaptarshiSarkar12

@KIET7UKE Where does the message comes, when the submit button is pressed?

@KIET7UKE
Copy link
Author

KIET7UKE commented Oct 9, 2022

@SaptarshiSarkar12 Yes, I forgot to mention that you have to just add your mail action tag
e.g: action = "mailto:yourmailid".

or

Just simply you can register here (https://getform.io/) & After that, you'll get an activation link just add in your HTML code and good to go.
drifty

You'll get this kind-of link just add in your action tag.
drifty1

@SaptarshiSarkar12
Copy link
Owner

@SaptarshiSarkar12 Yes, I forgot to mention that you have to just add your mail action tag e.g: action = "mailto:yourmailid".

Please add that

@KIET7UKE
Copy link
Author

KIET7UKE commented Oct 9, 2022

@SaptarshiSarkar12 I have created slides of steps on how to add this check below:
https://scribehow.com/shared/Get_message_from_visitors__xI1aPzEsR3SWdCcXGz0UWA

@SaptarshiSarkar12
Copy link
Owner

@SaptarshiSarkar12 I have created slides of steps on how to add this check below: https://scribehow.com/shared/Get_message_from_visitors__xI1aPzEsR3SWdCcXGz0UWA

@KIET7UKE Will this form work if I add "action=mailto:mail_ID"?

@KIET7UKE
Copy link
Author

KIET7UKE commented Oct 9, 2022

@SaptarshiSarkar12 I have created slides of steps on how to add this check below: https://scribehow.com/shared/Get_message_from_visitors__xI1aPzEsR3SWdCcXGz0UWA

@KIET7UKE Will this form work if I add "action=mailto:mail_ID"?

@SaptarshiSarkar12 Yes, It'll work but sometimes won't
The best case is to make an account at https://getform.io/
register here with your email address, and simply add the generated form endpoint in the action tag. easy.

@KIET7UKE
Copy link
Author

KIET7UKE commented Oct 9, 2022

@SaptarshiSarkar12 Did this all solved your problem?

@SaptarshiSarkar12
Copy link
Owner

@SaptarshiSarkar12 Did this all solved your problem?

No @KIET7UKE , getform.io allows upto 50 submissions. i researched and found that we can also use "php" file to send e-mail directly. Here is the article - https://blog.hubspot.com/marketing/html-form-email [method 2]

@KIET7UKE
Copy link
Author

KIET7UKE commented Oct 9, 2022

@SaptarshiSarkar12 Okay, so what are you saying, will I do that PHP connection also and then merge into this pr?

Or should I raise another PR for this contact form connection

@SaptarshiSarkar12
Copy link
Owner

@SaptarshiSarkar12 Okay, so what are you saying, will I do that PHP connection also and then merge into this pr?

Or should I raise another PR for this contact form connection

Will php work for github pages?

@SaptarshiSarkar12
Copy link
Owner

@SaptarshiSarkar12 Okay, so what are you saying, will I do that PHP connection also and then merge into this pr?
Or should I raise another PR for this contact form connection

Will php work for github pages?

@KIET7UKE

@KIET7UKE
Copy link
Author

KIET7UKE commented Oct 9, 2022

@SaptarshiSarkar12 Okay, so what are you saying, will I do that PHP connection also and then merge into this pr?

Or should I raise another PR for this contact form connection

Will php work for github pages?

That's I think will conflict in future

@SaptarshiSarkar12
Copy link
Owner

@SaptarshiSarkar12 Okay, so what are you saying, will I do that PHP connection also and then merge into this pr?
Or should I raise another PR for this contact form connection

Will php work for github pages?

That's I think will conflict in future

@KIET7UKE What about this : https://www.youtube.com/watch?v=0X0kovjzLrw ?

@KIET7UKE
Copy link
Author

KIET7UKE commented Oct 9, 2022

@SaptarshiSarkar12 Okay, so what are you saying, will I do that PHP connection also and then merge into this pr?
Or should I raise another PR for this contact form connection

Will php work for github pages?

That's I think will conflict in future

@KIET7UKE What about this : https://www.youtube.com/watch?v=0X0kovjzLrw ?

This is same as I did earlier with getforms.io

For now keep this things working, later on many contributors will improve this as well.

@SaptarshiSarkar12
Copy link
Owner

@SaptarshiSarkar12 Okay, so what are you saying, will I do that PHP connection also and then merge into this pr?
Or should I raise another PR for this contact form connection

Will php work for github pages?

That's I think will conflict in future

@KIET7UKE What about this : https://www.youtube.com/watch?v=0X0kovjzLrw ?

This is same as I did earlier with getforms.io

For now keep this things working, later on many contributors will improve this as well.

ok @KIET7UKE

@SaptarshiSarkar12
Copy link
Owner

@KIET7UKE Then, what to do with this PR?

@KIET7UKE
Copy link
Author

KIET7UKE commented Oct 9, 2022

@KIET7UKE Then, what to do with this PR?

@SaptarshiSarkar12 Merge it for now, will raise another issue for contact form redirection.
And I will work on this.

@SaptarshiSarkar12
Copy link
Owner

@KIET7UKE Then, what to do with this PR?

@SaptarshiSarkar12 Merge it for now, will raise another issue for contact form redirection. And I will work on this.

ok.

Copy link
Owner

@SaptarshiSarkar12 SaptarshiSarkar12 left a comment

Choose a reason for hiding this comment

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

@KIET7UKE Please commit the suggested changes

Website/index.html Show resolved Hide resolved
Website/index.html Show resolved Hide resolved
@KIET7UKE
Copy link
Author

KIET7UKE commented Oct 9, 2022

@SaptarshiSarkar12 done

@SaptarshiSarkar12
Copy link
Owner

@SaptarshiSarkar12 done

@KIET7UKE Don't you think that a form with no action doesn't seem good? I think that I shouldn't introduce the form section as the action hasn't been configured yet.

@KIET7UKE
Copy link
Author

KIET7UKE commented Oct 9, 2022

@SaptarshiSarkar12 Okay so give me some time, I'll make this form working and again commit it

@SaptarshiSarkar12
Copy link
Owner

@SaptarshiSarkar12 done

@KIET7UKE Don't you think that a form with no action doesn't seem good? I think that I shouldn't introduce the form section as the action hasn't been configured yet.

Also, "mailto:" opens the e-mail client , so , I think this issue needs more help from others.

@SaptarshiSarkar12
Copy link
Owner

@SaptarshiSarkar12 Okay so give me some time, I'll make this form working and again commit it

ok @KIET7UKE .

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
invalid This doesn't seem right
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants