Merge pull request #801 from zackproser/refactor-products-clean

Retire legacy Courses concepts, code, UI, data model

Author: zackproser

scripts/clean-og-images.ts

@@ -12,7 +12,6 @@ const OUTPUT_DIR = path.join(process.cwd(), 'public', 'og-images');
 const CONTENT_DIR = path.join(process.cwd(), 'src', 'content');
 
 // Access environment variables
-const COURSES_DISABLED = process.env.COURSES_DISABLED === 'true';
 
 // Get slugs from filesystem
 async function getContentSlugs(contentType: string): Promise<string[]> {
@@ -37,9 +36,6 @@ async function cleanOrphanedImages() {
 
   // Get all valid content slugs from different content types
   const contentTypes = ['blog', 'videos'];
-  if (!COURSES_DISABLED) {
-    contentTypes.push('learn/courses');
-  }
 
   // Get all valid slugs
   const validSlugs = (await Promise.all(

src/app/api/chat/route.ts

@@ -50,8 +50,8 @@ export async function POST(req: Request) {
     const blogPath = path.basename(blogUrl.replace('page.mdx', ''))
     const metadata = await getContentItemByDirectorySlug('blog', blogPath)
 
-    if (metadata) {
-      relatedBlogPosts.push({ ...metadata });
+    if (metadata && (metadata.type === 'blog' || metadata.type === 'video' || metadata.type === 'demo')) {
+      relatedBlogPosts.push(metadata as Blog);
     }
   }
   // Join all the chunks of text together, truncate to the maximum number of tokens, and return the result

src/app/api/webhooks/stripe/route.ts

@@ -3,7 +3,6 @@ import Stripe from 'stripe'
 import { PrismaClient } from '@prisma/client'
 import { sendReceiptEmail, SendReceiptEmailInput } from '@/lib/postmark'
 import { getContentItemByDirectorySlug } from '@/lib/content-handlers'
-import { COURSES_DISABLED } from '@/types'
 import { stripeLogger as logger } from '@/utils/logger' // Import centralized logger
 import { getContentUrlFromObject } from '@/lib/content-url'
 import { normalizeRouteOrFileSlug } from '@/lib/content-handlers'
@@ -162,14 +161,8 @@ export async function POST(req: Request) {
       logger.info('WEBHOOK: Fetching content details for slug:', slug)
       try {
         let content;
-        if (type === 'article' || type === 'blog') {
+        if (type === 'article' || type === 'blog' || type === 'video') {
           content = await getContentItemByDirectorySlug('blog', directorySlug)
-        } else if (type === 'course') {
-          logger.warn('WEBHOOK: Course purchase detected, using fallback content details as courses are disabled')
-          content = {
-            title: `Course: ${slug}`,
-            description: 'Premium Course Content'
-          };
         }
         logger.debug('WEBHOOK: Found content:', { title: content?.title })
         if (!content) {
@@ -204,10 +197,10 @@ export async function POST(req: Request) {
           TemplateModel: {
             CustomerName: user?.name || 'Valued Customer',
             ProductURL: productUrl,
-            ProductName: content?.title || `${type === 'article' ? 'Article' : 'Course'}: ${slug}`,
+            ProductName: content?.title || `Article: ${slug}`,
             Date: new Date().toLocaleDateString('en-US'),
             ReceiptDetails: {
-              Description: content?.description || `Premium ${type === 'article' ? 'Article' : 'Course'} Access`,
+              Description: content?.description || `Premium Article Access`,
               Amount: `$${session.amount_total! / 100}`,
               SupportURL: `${process.env.NEXT_PUBLIC_SITE_URL}/support`,
             },

src/app/blog/page.tsx

@@ -2,6 +2,7 @@ import { Metadata } from 'next'
 import BlogClient from './blog-client'
 import { createMetadata } from '@/utils/createMetadata'
 import { getAllContent } from '@/lib/content-handlers'
+import { Blog } from '@/types/content'
 
 // Base metadata using createMetadata
 const baseMetadata = createMetadata({
@@ -17,7 +18,7 @@ export const metadata: Metadata = {
 
 
 export default async function ArticlesIndex() {
-  const articles = await getAllContent('blog');
+  const articles = (await getAllContent('blog')).filter((a) => a.type === 'blog' || a.type === 'video' || a.type === 'demo') as Blog[];
 
   const years = [...new Set(articles.map((a) => new Date(a.date).getFullYear().toString()))].sort((a, b) => parseInt(b) - parseInt(a));
 

src/app/checkout/result/page.tsx

@@ -226,34 +226,6 @@ function CheckoutResultContent() {
             </div>
           </div>
         )}
-
-        {content.content.type === 'course' && (
-          <div className="mb-8">
-            <p className="mb-4">
-              Your course is now available. You can start learning right away!
-            </p>
-            {authStatus === 'authenticated' ? (
-              <Link
-                href={contentUrl as any}
-                className="inline-block px-4 py-2 bg-blue-500 text-white rounded hover:bg-blue-600 transition-colors"
-              >
-                Start Learning
-              </Link>
-            ) : (
-              <button
-                onClick={() => handleEmailSignIn(content.user.email, contentUrl)}
-                className="inline-block px-4 py-2 bg-blue-500 text-white rounded hover:bg-blue-600 transition-colors"
-              >
-                Sign in to Start Learning
-              </button>
-            )}
-            {authStatus === 'unauthenticated' && (
-              <p className="mt-2 text-sm text-gray-600">
-                You&apos;ll need to verify your email before accessing the full course.
-              </p>
-            )}
-          </div>
-        )}
       </div>
     </Container>
   );