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 email subscription button #1538

Merged
merged 10 commits into from
Jul 1, 2024
Merged

Add email subscription button #1538

merged 10 commits into from
Jul 1, 2024

Conversation

janhohenheim
Copy link
Collaborator

@janhohenheim janhohenheim commented Jun 29, 2024

Resolves #24

Don't merge this yet. It's unstyled and the mailchimp campaign is not yet done.

Previews:
image
image

@janhohenheim janhohenheim changed the title Do editorial pass on gunbug Add email subscription button Jun 29, 2024
@janhohenheim
Copy link
Collaborator Author

Currently waiting for a reply from MailChimp because of the following lovely automated emails:
image
image

I think we shouldn't be at fault here? Here's the relevant terms and conditions:
image

@janhohenheim
Copy link
Collaborator Author

Update: I simply created a new account and now it seems fine

@bash bash mentioned this pull request Jul 1, 2024
@janhohenheim janhohenheim marked this pull request as ready for review July 1, 2024 14:01
@janhohenheim
Copy link
Collaborator Author

Done!
@bash created this wonderful form for us:
344703367-6293402b-2998-4101-b7e1-2886232c69ac
Thanks @bash ❤️

Copy link
Collaborator

@Vrixyz Vrixyz left a comment

Choose a reason for hiding this comment

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

you're on 🔥 ; left a few nitpicks ; to be noted I'm not too qualified to judge html/CSS so it was mostly a wording pass.

janhohenheim and others added 2 commits July 1, 2024 16:20
@janhohenheim janhohenheim merged commit 22713e6 into source Jul 1, 2024
1 check passed
@janhohenheim janhohenheim deleted the email-signup branch July 1, 2024 14:50
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.

Email Subscription
4 participants