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

Add subscribe form (netlify) #111

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

dudil
Copy link
Contributor

@dudil dudil commented Jan 31, 2018

Description

I've added a form that will allow sites hosting on Netlify to set a subscribe (email) forms

Motivation and Context

I wanted to set up an email subscription list to my blog site in order to keep in touch with readers.
The Contact us didn't served for this purpose so I've added a new form for subscribe.

This Form is using the Netlify support to fire up a Zapier zap and send the subscribes to the preferred email service (in my case it is Emailoctopus)

How Has This Been Tested?

This change is already live on my Blog and working fine with the above integration.
(Netlify -> Zapier -> Emailoctopus)

While I could utilised the contact us form to be more generic, this approach will lead to code break with exist contact us forms.

! - I updated the exampleSite but didn't added this change into the description file.
If needed and approved, I'll be happy to do so

Hugo Version:
0.34

Browser(s):
Safari (iOS / OSX)

Screenshots (if appropriate):

Types of changes

  • Bug fix (non-breaking change which fixes an issue)
  • New feature (non-breaking change which adds functionality)
  • Breaking change (fix or feature that would cause existing functionality to change)

Checklist:

@pacollins
Copy link
Collaborator

Could you add a generic version too? Feel free to reference the contact form that checks if Netlify is enabled (via Front Matter) to use Netlify form handling and allows the user more freedom if they don't use Netlify?

@pacollins
Copy link
Collaborator

@dudil Any progress on this? No rush, I'm just checking all the PRs.

@dudil
Copy link
Contributor Author

dudil commented Feb 27, 2018

@pacollins - not yet.

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.

2 participants