From 091ce77c49faa6b629d381cbc8fa31845d3b97b4 Mon Sep 17 00:00:00 2001
From: Finley Ghosh <finley@springup.software>
Date: Mon, 12 Jan 2026 23:19:27 +1100
Subject: [PATCH] Ensuring pdf routes work

---
 go/cmd/server/main.go | 16 ++++++++--------
 1 file changed, 8 insertions(+), 8 deletions(-)

diff --git a/go/cmd/server/main.go b/go/cmd/server/main.go
index dd4a260f..7cad78a2 100644
--- a/go/cmd/server/main.go
+++ b/go/cmd/server/main.go
@@ -71,13 +71,6 @@ func main() {
 	// Static files
 	goRouter.PathPrefix("/static/").Handler(http.StripPrefix("/go/static/", http.FileServer(http.Dir("static"))))
 
-	// PDF files - use PDF_OUTPUT_DIR env var or fallback to relative path
-	pdfDir := os.Getenv("PDF_OUTPUT_DIR")
-	if pdfDir == "" {
-		pdfDir = "webroot/pdf"
-	}
-	goRouter.PathPrefix("/pdf/").Handler(http.StripPrefix("/go/pdf/", http.FileServer(http.Dir(pdfDir))))
-
 	// Quote routes
 	goRouter.HandleFunc("/quotes", quoteHandler.QuotesOutstandingView).Methods("GET")
 	goRouter.HandleFunc("/quotes/send-reminder", quoteHandler.SendManualReminder).Methods("POST")
@@ -89,7 +82,7 @@ func main() {
 	goRouter.HandleFunc("/attachments/{id}", attachmentHandler.Get).Methods("GET")
 	goRouter.HandleFunc("/attachments/{id}", attachmentHandler.Delete).Methods("DELETE")
 
-	// PDF generation endpoints - called from PHP app
+	// PDF generation routes
 	goRouter.HandleFunc("/pdf/generate-invoice", handlers.GenerateInvoicePDF).Methods("POST")
 	goRouter.HandleFunc("/pdf/generate-quote", handlers.GenerateQuotePDF).Methods("POST")
 	goRouter.HandleFunc("/pdf/generate-po", handlers.GeneratePurchaseOrderPDF).Methods("POST")
@@ -97,6 +90,13 @@ func main() {
 	goRouter.HandleFunc("/pdf/generate-orderack", handlers.GenerateOrderAckPDF).Methods("POST")
 	goRouter.HandleFunc("/pdf/count-pages", handlers.CountPages).Methods("POST")
 
+	// Serve generated PDFs
+	pdfDir := os.Getenv("PDF_OUTPUT_DIR")
+	if pdfDir == "" {
+		pdfDir = "webroot/pdf"
+	}
+	goRouter.PathPrefix("/pdf/").Handler(http.StripPrefix("/go/pdf/", http.FileServer(http.Dir(pdfDir))))
+
 	// The following routes are currently disabled:
 	/*
 		// API routes