Skip to content

Conversation

@rfay
Copy link
Member

@rfay rfay commented Oct 11, 2025

The Issue

We have obviously been putting newsletters here every month but don't have a 'newsletter' category, or any easy way to link to it.

How This PR Solves The Issue

Add it. Add the topic/category to existing newsletters.

Manual Testing Instructions

Visit /newsletter. Follow the link. Look at the newsletters.

Rendered: https://20251011-rfay-newsletter-cat.ddev-com-front-end.pages.dev/

See https://20251011-rfay-newsletter-cat.ddev-com-front-end.pages.dev/newsletter
Category listing at https://20251011-rfay-newsletter-cat.ddev-com-front-end.pages.dev/blog/category/newsletters/

Automated Testing Overview

Related Issue Link(s)

Release/Deployment Notes

@rfay rfay force-pushed the 20251011_rfay_newsletter_category branch from c91d8c2 to 737ad42 Compare October 11, 2025 17:22
@cloudflare-workers-and-pages
Copy link

cloudflare-workers-and-pages bot commented Oct 11, 2025

Deploying ddev-com-front-end with  Cloudflare Pages  Cloudflare Pages

Latest commit: 8317b26
Status: ✅  Deploy successful!
Preview URL: https://bc91d58d.ddev-com-front-end.pages.dev
Branch Preview URL: https://20251011-rfay-newsletter-cat.ddev-com-front-end.pages.dev

View logs

@rfay rfay requested a review from stasadev October 12, 2025 05:26
Copy link
Member

@stasadev stasadev left a comment

Choose a reason for hiding this comment

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

Good idea, looks good to me.

@stasadev stasadev changed the title Add 'Newletter' category and put newsletters in it Add 'Newsletter' category and put newsletters in it Oct 13, 2025
@stasadev stasadev merged commit 2dcedde into main Oct 13, 2025
5 checks passed
@stasadev stasadev deleted the 20251011_rfay_newsletter_category branch October 13, 2025 10: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.

3 participants