improve delivery reliability

Author: Thomas Stromberg

email/templates.go

@@ -21,7 +21,8 @@ func (s *Sender) formatNotificationBody(sub *notifier.Subscription, thread *noti
 	b.WriteString(".post:last-of-type { border-bottom: none; }\n")
 	b.WriteString(".author { color: #e67e22; font-weight: 600; }\n")
 	b.WriteString(".timestamp { color: #7f8c8d; font-size: 0.9em; }\n")
-	b.WriteString(".content { background: #f8f9fa; padding: 20px; border-radius: 8px; margin: 15px 0; white-space: pre-wrap; }\n")
+	b.WriteString(".content { background: #f8f9fa; padding: 20px; border-radius: 8px; margin: 15px 0; }\n")
+	b.WriteString(".content img { max-width: 100%; height: auto; margin: 10px 0; display: block; }\n")
 	b.WriteString(".footer { margin-top: 20px; padding-top: 10px; border-top: 2px solid #ecf0f1; color: #7f8c8d; font-size: 0.9em; }\n")
 	b.WriteString("a { color: #e67e22; text-decoration: none; }\n")
 	b.WriteString("a:hover { text-decoration: underline; }\n")
@@ -50,7 +51,12 @@ func (s *Sender) formatNotificationBody(sub *notifier.Subscription, thread *noti
 		b.WriteString("</div>\n")
 
 		b.WriteString("<div class=\"content\">\n")
-		b.WriteString(escapeHTML(post.Content))
+		// Use HTML content if available (includes images), otherwise fall back to plain text
+		if post.HTMLContent != "" {
+			b.WriteString(post.HTMLContent)
+		} else {
+			b.WriteString(escapeHTML(post.Content))
+		}
 		b.WriteString("</div>\n")
 
 		b.WriteString(fmt.Sprintf("<a href=\"%s\" class=\"view-post\">View this post on ADVRider</a>\n", escapeHTML(post.URL)))

main.go

@@ -87,6 +87,14 @@ func main() {
 		storageSvc := storage.New(nil, "", localStorage, []byte(salt), logger)
 		pollSvc := poll.New(scraperSvc, storageSvc, emailSender, logger)
 
+		// Run initial polling cycle on startup
+		logger.Info("Running initial polling cycle on startup")
+		if err := pollSvc.CheckAll(ctx); err != nil {
+			logger.Warn("Initial polling cycle failed", "error", err)
+		} else {
+			logger.Info("Initial polling cycle completed successfully")
+		}
+
 		// Create and run server
 		srv := server.New(&server.Config{
 			Scraper:    scraperSvc,
@@ -149,6 +157,14 @@ func main() {
 	storageSvc := storage.New(storageClient, bucket, "", []byte(salt), logger)
 	pollSvc := poll.New(scraperSvc, storageSvc, emailSender, logger)
 
+	// Run initial polling cycle on startup
+	logger.Info("Running initial polling cycle on startup")
+	if err := pollSvc.CheckAll(ctx); err != nil {
+		logger.Warn("Initial polling cycle failed", "error", err)
+	} else {
+		logger.Info("Initial polling cycle completed successfully")
+	}
+
 	// Create server
 	srv := server.New(&server.Config{
 		Scraper:    scraperSvc,

media/style.css

@@ -61,7 +61,26 @@ strong {
 }
 
 a {
-	display: inline-block;
+	color: #e67e22;
+	text-decoration: none;
+	transition: color 0.2s ease;
+}
+
+a:hover {
+	text-decoration: underline;
+}
+
+a:focus {
+	outline: 2px solid #e67e22;
+	outline-offset: 2px;
+	text-decoration: underline;
+}
+
+/* Button-styled links for CTAs */
+a.button {
+	display: inline-flex;
+	align-items: center;
+	justify-content: center;
 	background: #e67e22;
 	color: white;
 	padding: 14px 28px;
@@ -73,18 +92,16 @@ a {
 	box-shadow: 0 4px 12px rgba(230, 126, 34, 0.3);
 	margin-top: 16px;
 	min-height: 44px;
-	display: inline-flex;
-	align-items: center;
-	justify-content: center;
 }
 
-a:hover {
+a.button:hover {
 	background: #d35400;
 	transform: translateY(-1px);
 	box-shadow: 0 6px 20px rgba(230, 126, 34, 0.4);
+	text-decoration: none;
 }
 
-a:focus {
+a.button:focus {
 	outline: 3px solid #e67e22;
 	outline-offset: 2px;
 }
@@ -187,35 +204,17 @@ button.secondary:focus {
 .thread-url {
 	font-family: 'SF Mono', 'Monaco', 'Courier New', monospace;
 	font-size: 14px;
-	color: #666;
 	word-break: break-all;
 	margin-bottom: 8px;
 }
 
 .thread-url a {
-	display: inline;
-	background: none;
 	color: #0066cc;
 	text-decoration: underline;
-	padding: 2px;
-	margin: 0;
-	box-shadow: none;
-	font-size: inherit;
-	font-weight: normal;
-	border-radius: 3px;
 }
 
 .thread-url a:hover {
 	color: #004499;
-	background: none;
-	transform: none;
-	box-shadow: none;
-}
-
-.thread-url a:focus {
-	outline: 2px solid #0066cc;
-	outline-offset: 2px;
-	background: rgba(0, 102, 204, 0.1);
 }
 
 .thread-meta {
@@ -250,24 +249,7 @@ button.secondary:focus {
 	margin-bottom: 24px;
 }
 
-.empty a {
-	display: inline-block;
-	background: #e67e22;
-	color: white;
-	padding: 14px 28px;
-	border-radius: 8px;
-	text-decoration: none;
-	font-size: 17px;
-	font-weight: 600;
-	box-shadow: 0 4px 12px rgba(230, 126, 34, 0.3);
-	margin-top: 8px;
-}
-
-.empty a:hover {
-	background: #d35400;
-	transform: translateY(-1px);
-	box-shadow: 0 6px 20px rgba(230, 126, 34, 0.4);
-}
+/* Empty state uses button class for CTA */
 
 .logo {
 	text-align: center;
@@ -354,32 +336,6 @@ button:active {
 	color: #767676;
 }
 
-.footer a {
-	display: inline;
-	background: none;
-	color: #e67e22;
-	text-decoration: none;
-	padding: 2px 4px;
-	margin: 0 2px;
-	box-shadow: none;
-	font-size: inherit;
-	font-weight: normal;
-	border-radius: 3px;
-}
-
-.footer a:hover {
-	text-decoration: underline;
-	background: none;
-	transform: none;
-	box-shadow: none;
-}
-
-.footer a:focus {
-	outline: 2px solid #e67e22;
-	outline-offset: 2px;
-	text-decoration: underline;
-}
-
 @media (max-width: 600px) {
 	.container {
 		padding: 32px 24px;

pkg/notifier/notifier.go

@@ -5,11 +5,12 @@ import "time"
 
 // Post represents a single post in a thread.
 type Post struct {
-	ID        string
-	Author    string
-	Content   string
-	Timestamp string
-	URL       string
+	ID          string
+	Author      string
+	Content     string // Plain text content for fallback
+	HTMLContent string // HTML content with images and formatting
+	Timestamp   string
+	URL         string
 }
 
 // Thread represents a monitored thread with its state.