Skip to content

feat(blog): reaction milestone emails, unified notification mailer, and mobile UI refinements - #163

Merged
trtajim merged 4 commits into
mainfrom
feat/blog-reaction-milestone-notifications
Aug 25, 2026
Merged

feat(blog): reaction milestone emails, unified notification mailer, and mobile UI refinements#163
trtajim merged 4 commits into
mainfrom
feat/blog-reaction-milestone-notifications

Conversation

@trtajim

@trtajim trtajim commented Aug 25, 2026

Copy link
Copy Markdown
Member

Summary

This PR introduces reaction milestone email notifications, consolidates all blog email notifications into a single reusable class, and refines the mobile UI on blog pages.

Key Changes:

  1. Milestone-Based Reaction Notifications:

    • Sends notification emails to blog authors only at key milestones: 1 (first reaction), 10, 25, 50, 100, 250, 500, 1000, and every 500 thereafter.
    • Prevents email spam while keeping authors engaged when their posts reach major audience milestones.
  2. Unified Notification Mailer:

    • Replaced separate mail classes with a single reusable BlogNotificationMail with static factories:
      • BlogNotificationMail::forComment($blog, $comment)
      • BlogNotificationMail::forReactionMilestone($blog, $reactor, $count)
    • Reuses the standard emails.bulk_announcement Blade template.
  3. Mobile & UX Refinements:

    • Comment Deletion on Touch Screens: Made the delete button always visible on mobile viewports (opacity-100 sm:opacity-0 sm:group-hover:opacity-100) so mobile users can easily delete their comments without hover states.
    • Minimal "Write for Us" Callout: Replaced the bulky full-width CTA with a sleek, compact 1-line callout.
  4. Testing & QA:

    • Added Pest feature tests for milestone notifications and self-reaction suppression.
    • 57 passing tests (193 assertions) across PHP 8.3, 8.4, 8.5 with all formatters and linters passing.

@trtajim trtajim changed the title feat(blog): send email notifications on reaction milestones feat(blog): reaction milestone emails, unified notification mailer, and mobile UI refinements Aug 25, 2026
@trtajim
trtajim merged commit 80b142c into main Aug 25, 2026
5 checks passed
@trtajim
trtajim deleted the feat/blog-reaction-milestone-notifications branch August 25, 2026 17:27
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.

1 participant