Skip to content

Add Octopus Newsletter Form #445

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.

Sign up for GitHub

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

Jump to bottom
Merged
merged 5 commits into from
Sep 19, 2025
Merged

Add Octopus Newsletter Form #445

merged 5 commits into from
Sep 19, 2025

Conversation

@richarddushime
Copy link
Contributor

@richarddushime richarddushime commented Sep 14, 2025 •
edited
Loading

Description

Fixes # (issue)

Type of Change

  • Documentation/Contents/grammar update
  • New feature (non-breaking change which adds functionality)

How Has This Been Tested?

  • Tested Locally
  • Manual review / previewed on staging.forrt.org content/webpage changes

Checklist for Content Editors and Non-Developers

  • The content is clear, accurate, and follows community guidelines.
  • All updated content has been previewed on the staging site.
  • All links, references, and formatting have been checked for correctness.
  • The change aligns with the overall style and communication goals.
  • No broken links in text/content

Checklist for Developers:

  • I have attempted to stay aligned to related code in this repository rather than reinventing the wheel.
  • I have performed a self-review of my own code.
  • I have commented my code, particularly in hard-to-understand areas.
  • I have made corresponding changes to the documentation.
  • My changes generate no new warnings.

Additional Notes

@richarddushime richarddushime requested a review from a team as a code owner September 14, 2025 09:50
@github-actions
Copy link
Contributor

github-actions bot commented Sep 14, 2025

🤖 Hi @richarddushime, I've received your request, and I'm working on it now! You can track my progress in the logs for more details.

@github-actions
Copy link
Contributor

github-actions bot commented Sep 14, 2025

👍 All image files/references (if any) are in webp format, in line with our policy.

@richarddushime richarddushime changed the title Newsletter Updates [Draft] Newsletter Updates Sep 14, 2025
github-actions[bot]
github-actions bot reviewed Sep 14, 2025
View reviewed changes
Copy link
Contributor

@github-actions github-actions bot left a comment

Choose a reason for hiding this comment

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

📋 Review Summary

This pull request replaces the Mailchimp subscription form with a new one from EmailOctopus. The changes are simple and clear. I've added one minor suggestion to improve the comment's descriptiveness.

🔍 General Feedback

  • The switch to EmailOctopus is well-implemented.
  • Consider keeping code comments updated and descriptive for future maintainability.

@richarddushime @github-actions
Update layouts/shortcodes/subscribe.html
0bcb825 
Co-authored-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com>
@LukasWallrich
Copy link
Contributor

LukasWallrich commented Sep 14, 2025

✅ This PR has been deployed to staging as part of an aggregated deployment. View at: https://staging.forrt.org

11 similar comments
@LukasWallrich
Copy link
Contributor

LukasWallrich commented Sep 14, 2025

✅ This PR has been deployed to staging as part of an aggregated deployment. View at: https://staging.forrt.org

@LukasWallrich
Copy link
Contributor

LukasWallrich commented Sep 14, 2025

✅ This PR has been deployed to staging as part of an aggregated deployment. View at: https://staging.forrt.org

@LukasWallrich
Copy link
Contributor

LukasWallrich commented Sep 14, 2025

✅ This PR has been deployed to staging as part of an aggregated deployment. View at: https://staging.forrt.org

@LukasWallrich
Copy link
Contributor

LukasWallrich commented Sep 14, 2025

✅ This PR has been deployed to staging as part of an aggregated deployment. View at: https://staging.forrt.org

@LukasWallrich
Copy link
Contributor

LukasWallrich commented Sep 14, 2025

✅ This PR has been deployed to staging as part of an aggregated deployment. View at: https://staging.forrt.org

@LukasWallrich
Copy link
Contributor

LukasWallrich commented Sep 14, 2025

✅ This PR has been deployed to staging as part of an aggregated deployment. View at: https://staging.forrt.org

@LukasWallrich
Copy link
Contributor

LukasWallrich commented Sep 14, 2025

✅ This PR has been deployed to staging as part of an aggregated deployment. View at: https://staging.forrt.org

@LukasWallrich
Copy link
Contributor

LukasWallrich commented Sep 14, 2025

✅ This PR has been deployed to staging as part of an aggregated deployment. View at: https://staging.forrt.org

@LukasWallrich
Copy link
Contributor

LukasWallrich commented Sep 14, 2025

✅ This PR has been deployed to staging as part of an aggregated deployment. View at: https://staging.forrt.org

@LukasWallrich
Copy link
Contributor

LukasWallrich commented Sep 15, 2025

✅ This PR has been deployed to staging as part of an aggregated deployment. View at: https://staging.forrt.org

@LukasWallrich
Copy link
Contributor

LukasWallrich commented Sep 15, 2025

✅ This PR has been deployed to staging as part of an aggregated deployment. View at: https://staging.forrt.org

@LukasWallrich
Copy link
Contributor

LukasWallrich commented Sep 18, 2025

✅ This PR has been deployed to staging as part of an aggregated deployment. View at: https://staging.forrt.org

@richarddushime richarddushime changed the title [Draft] Newsletter Updates Add Octopus Newsletter Form Sep 19, 2025
@LukasWallrich
Copy link
Contributor

LukasWallrich commented Sep 19, 2025

✅ This PR has been deployed to staging as part of an aggregated deployment. View at: https://staging.forrt.org

LukasWallrich
LukasWallrich approved these changes Sep 19, 2025
View reviewed changes
Copy link
Contributor

@LukasWallrich LukasWallrich left a comment

Choose a reason for hiding this comment

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

Good to go.

@LukasWallrich LukasWallrich merged commit 9d42bf0 into master Sep 19, 2025
14 checks passed
@LukasWallrich LukasWallrich deleted the mewsletter branch September 19, 2025 12:58
richarddushime added a commit that referenced this pull request Nov 13, 2025
@richarddushime @github-actions @LukasWallrich
Add Octopus Newsletter Form (#445)
5037cb5 
* Newsletter Updates

* Update layouts/shortcodes/subscribe.html

Co-authored-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com>

* Fixes

---------

Co-authored-by: github-actions[bot] <github-actions[bot]@users.noreply.github.com>
Co-authored-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com>
Co-authored-by: Lukas Wallrich <lukas.wallrich@gmail.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@github-actions github-actions[bot] github-actions[bot] left review comments

@LukasWallrich LukasWallrich LukasWallrich approved these changes

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

3 participants

@richarddushime @LukasWallrich