Only logged in users can create topics

bknoles · bknoles · commit 50e2bc6a6b22 · 2025-11-21T20:18:18.000-08:00
diff --git a/app/controllers/topics_controller.rb b/app/controllers/topics_controller.rb
@@ -1,4 +1,6 @@
 class TopicsController < ApplicationController
+  before_action :verify_user, except: [ :index, :show ]
+
   def index
     topics = Topic.all
       .includes(:user, :category, messages: :user)
@@ -45,4 +47,8 @@ def create
   def topic_params
     params.require(:topic).permit(:title, :category_id, messages_attributes: [ :body ])
   end
+
+  def verify_user
+    redirect_to topics_path unless current_user
+  end
 end
diff --git a/app/frontend/pages/categories/show.tsx b/app/frontend/pages/categories/show.tsx
@@ -1,22 +1,24 @@
 import { Link } from "@inertiajs/react";
 import TopicsTable from "../../components/TopicsTable"
 import AppLayout from "../../layouts/AppLayout"
-import { Category } from "../../types"
+import { Category, User } from "../../types"
 
-function CategoriesShow({ category }: { category: Category }) {
+function CategoriesShow({ category, current_user: currentUser }: { category: Category, current_user: User }) {
   const { topics } = category;
 
   return (
     <AppLayout>
       <div className="flex items-center justify-between mb-6">
         <h1 className="text-2xl font-bold text-gray-900">Category: {category.name}</h1>
-        <Link
+        {currentUser && (
+          <Link
           href={'/topics/new'}
           data={{topic: { category_id: category.id }}}
           className="bg-sky-600 hover:bg-sky-700 text-white px-4 py-2 rounded-lg text-sm font-medium transition-colors shadow inline-block text-center"
         >
-          New Topic
-        </Link>
+            New Topic
+          </Link>
+        )}
       </div>
 
       <div className="max-w-7xl mx-auto">
diff --git a/app/frontend/pages/my_topics/index.tsx b/app/frontend/pages/my_topics/index.tsx
@@ -1,21 +1,23 @@
 import { Head, Link } from '@inertiajs/react'
 import AppLayout from '../../layouts/AppLayout'
-import { Topic } from '../../types'
+import { Topic, User } from '../../types'
 import TopicsTable from '../../components/TopicsTable'
 
-function MyTopicsIndex({ topics }: { topics: Topic[] }) {
+function MyTopicsIndex({ topics, current_user: currentUser }: { topics: Topic[], current_user: User }) {
   return (
     <AppLayout>
       <Head title="My Topics - Pups & Pourovers" />
 
       <div className="flex items-center justify-between mb-6">
         <h1 className="text-2xl font-bold text-gray-900">My Topics</h1>
-        <Link
+        {currentUser && (
+          <Link
           href="/topics/new"
           className="bg-sky-600 hover:bg-sky-700 text-white px-4 py-2 rounded-lg text-sm font-medium transition-colors shadow inline-block text-center"
         >
-          New Topic
-        </Link>
+            New Topic
+          </Link>
+        )}
       </div>
 
       <div className="max-w-7xl mx-auto">
diff --git a/app/frontend/pages/topics/index.tsx b/app/frontend/pages/topics/index.tsx
@@ -1,21 +1,23 @@
 import { Head, Link } from '@inertiajs/react'
 import AppLayout from '../../layouts/AppLayout'
-import { Topic } from '../../types'
+import { Topic, User } from '../../types'
 import TopicsTable from '../../components/TopicsTable'
 
-function TopicsIndex({ topics }: { topics: Topic[] }) {
+function TopicsIndex({ topics, current_user: currentUser }: { topics: Topic[], current_user: User }) {
   return (
     <AppLayout>
       <Head title="Topics - Pups & Pourovers" />
 
       <div className="max-w-7xl mx-auto">
         <div className="flex justify-end mb-6">
-          <Link
+          {currentUser && (
+            <Link
             href="/topics/new"
             className="bg-sky-600 hover:bg-sky-700 text-white px-4 py-2 rounded-lg text-sm font-medium transition-colors shadow inline-block text-center"
           >
-            New Topic
-          </Link>
+              New Topic
+            </Link>
+          )}
         </div>
         <div className="bg-white shadow-sm rounded-lg overflow-hidden">
           <TopicsTable topics={topics} />
