From f2264eba167327c1327baea615cb3f3f84d1895e Mon Sep 17 00:00:00 2001
From: Josh <jcoakley@aptivaai.com>
Date: Tue, 22 Jul 2025 19:25:19 +0000
Subject: [PATCH] Added button for New College Plan

---
 src/components/CareerSelectDropdown.js |   4 +-
 src/components/CollegeProfileForm.js   |  10 --
 src/components/CollegeProfileList.js   | 131 +++++++++++++++++++++----
 3 files changed, 117 insertions(+), 28 deletions(-)

diff --git a/src/components/CareerSelectDropdown.js b/src/components/CareerSelectDropdown.js
index 5efcafa..ac8ba7a 100644
--- a/src/components/CareerSelectDropdown.js
+++ b/src/components/CareerSelectDropdown.js
@@ -26,7 +26,9 @@ const CareerSelectDropdown = ({ existingCareerProfiles, selectedCareer, onChange
 
   return (
     <div className="career-select-dropdown">
-      <label>Select Career Path:</label>
+     <label className="block font-medium mb-1">
+        Select the career this college plan belongs to:
+      </label>
       {loading ? (
         <p>Loading career paths...</p>
       ) : (
diff --git a/src/components/CollegeProfileForm.js b/src/components/CollegeProfileForm.js
index f0e840a..2894bc2 100644
--- a/src/components/CollegeProfileForm.js
+++ b/src/components/CollegeProfileForm.js
@@ -240,10 +240,6 @@ return (
     <h2 className="text-2xl font-semibold">
       {id === 'new' ? 'New' : 'Edit'} College Plan
     </h2>
-
-    {(form.college_enrollment_status === 'currently_enrolled' ||
-      form.college_enrollment_status === 'prospective_student') ? (
-      /* ───────────────────────────────────────────────────────── */
       <div className="space-y-4">
 
         {/* 1 │ Location / modality check‑boxes */}
@@ -492,12 +488,6 @@ return (
           />
         </div>
       </div>
-    ) : (
-      <p>
-        User is neither currently enrolled nor a prospective student – nothing to
-        edit.
-      </p>
-    )}
 
     {/* 11 │ Action buttons */}
     <div className="pt-4">
diff --git a/src/components/CollegeProfileList.js b/src/components/CollegeProfileList.js
index aabc834..84c0d82 100644
--- a/src/components/CollegeProfileList.js
+++ b/src/components/CollegeProfileList.js
@@ -1,23 +1,100 @@
-import React, { useEffect, useState } from 'react';
-import { Link, useNavigate } from 'react-router-dom';
+// src/components/CollegeProfileList.js
+import React, { useEffect, useState } from "react";
+import { Link, useNavigate, useParams } from "react-router-dom";
+import CareerSelectDropdown from "./CareerSelectDropdown.js";
+import authFetch from "../utils/authFetch.js";
 
 export default function CollegeProfileList() {
-  const [rows, setRows] = useState([]);
-  const nav   = useNavigate();
-  const token = localStorage.getItem('token');
+  const { careerId } = useParams();                // may be undefined
+  const navigate = useNavigate();
+  const token = localStorage.getItem("token");
 
+  /* ───────── existing lists ───────── */
+  const [rows, setRows] = useState([]);
+  const [careerRows, setCareerRows] = useState([]);
+
+  /* ───────── ui state ───────── */
+  const [showPicker, setShowPicker] = useState(false);
+  const [loadingCareers, setLoadingCareers] = useState(true);
+
+  /* ───────── load college plans ───────── */
   useEffect(() => {
-    fetch('/api/premium/college-profile/all', {
+    fetch("/api/premium/college-profile/all", {
       headers: { Authorization: `Bearer ${token}` }
     })
-      .then(r => r.json())
-      .then(d => setRows(d.collegeProfiles || []));
+      .then((r) => r.json())
+      .then((d) => setRows(d.collegeProfiles || []));
   }, [token]);
 
-  return (
-    <div className="max-w-5xl mx-auto space-y-4">
-      <h2 className="text-2xl font-semibold">College Profiles</h2>
+  /* ───────── load career profiles for the picker ───────── */
+  useEffect(() => {
+    (async () => {
+      try {
+        const res = await authFetch("/api/premium/career-profile/all");
+        const data = await res.json();
+        setCareerRows(data.careerProfiles || []);
+      } catch (err) {
+        console.error("Career profiles load failed:", err);
+      } finally {
+        setLoadingCareers(false);
+      }
+    })();
+  }, []);
 
+  /* ───────── delete helper ───────── */
+  async function handleDelete(id) {
+    if (!window.confirm("Delete this college plan?")) return;
+    try {
+      await fetch(`/api/premium/college-profile/${id}`, {
+        method: "DELETE",
+        headers: { Authorization: `Bearer ${token}` }
+      });
+      setRows((r) => r.filter((row) => row.id !== id));
+    } catch (err) {
+      console.error("Delete failed:", err);
+      alert("Could not delete – see console.");
+    }
+  }
+
+  return (
+    <div className="max-w-5xl mx-auto space-y-6">
+      {/* ───────── header row ───────── */}
+      <div className="flex items-center justify-between">
+        <h2 className="text-2xl font-semibold">College Plans</h2>
+
+        {/* new‑plan button & inline picker */}
+        {!showPicker ? (
+          <button
+            onClick={() => setShowPicker(true)}
+            className="px-3 py-2 bg-blue-600 text-white rounded"
+          >
+            + New College Plan
+          </button>
+        ) : (
+          <div className="p-4 border rounded bg-gray-50 max-w-md">
+            <CareerSelectDropdown
+              existingCareerProfiles={careerRows}
+              selectedCareer={null}
+              loading={loadingCareers}
+              authFetch={authFetch}
+              onChange={(careerObj) => {
+                if (!careerObj?.id) return;
+                navigate(`/profile/college/${careerObj.id}/new`);
+              }}
+            />
+            <div className="mt-2 text-right">
+              <button
+                onClick={() => setShowPicker(false)}
+                className="text-sm text-gray-600 underline"
+              >
+                cancel
+              </button>
+            </div>
+          </div>
+        )}
+      </div>
+
+      {/* ───────── table of existing college plans ───────── */}
       <table className="w-full border text-sm">
         <thead className="bg-gray-100">
           <tr>
@@ -28,27 +105,47 @@ export default function CollegeProfileList() {
             <th className="p-2"></th>
           </tr>
         </thead>
+
         <tbody>
-          {rows.map(r => (
+          {rows.map((r) => (
             <tr key={r.id} className="border-t">
               <td className="p-2">{r.career_title}</td>
               <td className="p-2">{r.selected_school}</td>
               <td className="p-2">{r.selected_program}</td>
-              <td className="p-2">{r.created_at?.slice(0,10)}</td>
-              <td className="p-2">
+              <td className="p-2">{r.created_at?.slice(0, 10)}</td>
+              <td className="p-2 space-x-2 whitespace-nowrap">
                 <Link
                   to={`/profile/college/${r.career_profile_id}/${r.id}`}
                   className="underline text-blue-600"
                 >
                   edit
                 </Link>
+                <button
+                  onClick={() => handleDelete(r.id)}
+                  className="underline text-red-600"
+                >
+                  delete
+                </button>
               </td>
             </tr>
           ))}
+
           {rows.length === 0 && (
-            <tr><td colSpan={5} className="p-4 text-center text-gray-500">
-              No college profiles yet
-            </td></tr>
+            <tr>
+              <td colSpan={6} className="p-8 text-center text-gray-500">
+                No college profiles yet.&nbsp;
+                <button
+                  className="text-blue-600 underline"
+                  onClick={() =>
+                    careerId
+                      ? navigate(`/profile/college/${careerId}/new`)
+                      : setShowPicker(true)
+                  }
+                >
+                  Create one now.
+                </button>
+              </td>
+            </tr>
           )}
         </tbody>
       </table>