Update a.jsx

a.jsx

@@ -1565,7 +1565,7 @@ function CountdownBadge({ remaining, urgent, warning }) {
 }
 
 // ── TestingProfilesPortal ─────────────────────────────────────────────────────
-function TestingProfilesPortal({ currentUser, onEdit, onBack, onLogout }) {
+function TestingProfilesPortal({ currentUser, currentRole, onEdit, onBack, onLogout }) {
   const [profiles,     setProfiles]     = useState([]);
   const [loading,      setLoading]      = useState(true);
   const [deleteTarget, setDeleteTarget] = useState(null);
@@ -1687,7 +1687,7 @@ function TestingProfilesPortal({ currentUser, onEdit, onBack, onLogout }) {
 }
 
 // ── ProfileEditorPortal ───────────────────────────────────────────────────────
-function ProfileEditorPortal({ profile: initialProfile, currentUser, onBack, onLogout }) {
+function ProfileEditorPortal({ profile: initialProfile, currentUser, currentRole, onBack, onLogout }) {
   const [profile, setProfile] = useState(initialProfile);
   const [qs,      setQs]      = useState(() => initialProfile.questions ?? []);
   const [dirty,   setDirty]   = useState(false);
@@ -1971,7 +1971,7 @@ function ProfilePickerPortal({ mode, session, onPick, onCancel, onLogout }) {
 }
 
 // ── ProductsPortal ────────────────────────────────────────────────────────────
-function ProductsPortal({ currentUser, onBack, onLogout }) {
+function ProductsPortal({ currentUser, currentRole, onBack, onLogout }) {
   const [products,     setProducts]     = useState([]);
   const [loading,      setLoading]      = useState(true);
   const [deleteTarget, setDeleteTarget] = useState(null);
@@ -2116,7 +2116,7 @@ const ROLE_DESC   = {
   tester:   "Tester portal only — upload screenshots per session",
 };
 
-function AccountsPortal({ currentUser, onBack, onLogout }) {
+function AccountsPortal({ currentUser, currentRole, onBack, onLogout }) {
   const [users,        setUsers]        = useState([]);
   const [allProducts,  setAllProducts]  = useState([]);
   const [loading,      setLoading]      = useState(true);
@@ -2519,19 +2519,21 @@ function KanbanPortal({ currentUser, currentRole, onBack, onLogout, onNavigate }
 
   const openEdit = (task) => setModal({ mode: "edit", task: { ...task } });
 
-  const handleSave = async (task) => {
+  const handleSave = (task) => {
     const orig = tasks.find(t => t.id === task.id);
     task.updatedAt = Date.now();
     if (task.label === "daily" && task.status === "backlog") task.status = "in-progress";
-    await dbSaveTask(task);
 
-    // Close modal and update UI immediately — don't wait for notifications
+    // Optimistic update — show card and close modal instantly
     setTasks(prev => {
       const idx = prev.findIndex(t => t.id === task.id);
       return idx >= 0 ? prev.map(t => t.id === task.id ? task : t) : [...prev, task];
     });
     setModal(null);
 
+    // Persist to DB in background
+    dbSaveTask(task).catch(() => {});
+
     // Activity log + notifications fire in background
     const isNew = !orig;
     const changes = [];
@@ -2636,7 +2638,7 @@ function KanbanPortal({ currentUser, currentRole, onBack, onLogout, onNavigate }
         </div>
         <div className="header-right">
           {canCreate && <button className="btn-view-report" onClick={openCreate}>+ New Task</button>}
-          <button className="btn-secondary-sm" onClick={() => setShowLog(true)}>📋 Log</button>
+          {isAdmin && <button className="btn-secondary-sm" onClick={() => setShowLog(true)}>📋 Log</button>}
           <button className="btn-secondary-sm" onClick={onBack}>← Portal</button>
           <NotificationBell username={currentUser} onNavigate={onNavigate} />
           <ProfileMenu username={currentUser} role={currentRole} onLogout={onLogout} />
@@ -2820,12 +2822,10 @@ function TaskModal({ task: initTask, mode, currentUser, currentRole, products, a
     (isTester || task.productId) &&
     (isTester || (task.label === "daily" ? !!task.recurTime : !!task.dueDate));
 
-  const handleSubmit = async (e) => {
+  const handleSubmit = (e) => {
     e.preventDefault();
     if (!isValid) return;
-    setSaving(true);
-    await onSave(task);
-    setSaving(false);
+    onSave(task);
   };
 
   return (
@@ -2992,8 +2992,8 @@ function TaskModal({ task: initTask, mode, currentUser, currentRole, products, a
 
           <div className="modal-actions" style={{ margin: "16px 28px 24px" }}>
             <button type="button" className="btn-cancel" onClick={onClose}>Cancel</button>
-            <button type="submit" className="btn-primary" disabled={saving || !isValid}>
-              {saving ? "Saving…" : mode === "create" ? "Create Task" : "Save Changes"}
+            <button type="submit" className="btn-primary" disabled={!isValid}>
+              {mode === "create" ? "Create Task" : "Save Changes"}
             </button>
           </div>
         </form>
@@ -3068,18 +3068,20 @@ function TicketsPortal({ currentUser, currentRole, onBack, onLogout, onNavigate
     setModal({ mode: "edit", ticket: { ...ticket } });
   };
 
-  const handleSave = async (ticket) => {
+  const handleSave = (ticket) => {
     const orig = tickets.find(t => t.id === ticket.id);
     ticket.updatedAt = Date.now();
-    await dbSaveTicket(ticket);
 
-    // Close modal and update UI immediately — don't wait for notifications
+    // Optimistic update — show ticket and close modal instantly
     setTickets(prev => {
       const idx = prev.findIndex(t => t.id === ticket.id);
       return idx >= 0 ? prev.map(t => t.id === ticket.id ? ticket : t) : [...prev, ticket];
     });
     setModal(null);
 
+    // Persist to DB in background
+    dbSaveTicket(ticket).catch(() => {});
+
     // Activity log + notification fire in background
     const isNew = !orig;
     const changes = [];
@@ -3171,7 +3173,7 @@ function TicketsPortal({ currentUser, currentRole, onBack, onLogout, onNavigate
         <div className="header-left"><h1>Tickets</h1></div>
         <div className="header-right">
           <button className="btn-view-report" onClick={openCreate}>+ New Ticket</button>
-          <button className="btn-secondary-sm" onClick={() => setShowLog(true)}>📋 Log</button>
+          {isAdmin && <button className="btn-secondary-sm" onClick={() => setShowLog(true)}>📋 Log</button>}
           <button className="btn-secondary-sm" onClick={onBack}>← Portal</button>
           <NotificationBell username={currentUser} onNavigate={onNavigate} />
           <ProfileMenu username={currentUser} role={currentRole} onLogout={onLogout} />
@@ -3293,12 +3295,10 @@ function TicketModal({ ticket: initTicket, mode, currentUser, currentRole, produ
   const canAssign       = isAdmin || isReviewer;
   const canChangeStatus = isAdmin || isReviewer;
 
-  const handleSubmit = async (e) => {
+  const handleSubmit = (e) => {
     e.preventDefault();
     if (!ticket.title.trim()) return;
-    setSaving(true);
-    await onSave(ticket);
-    setSaving(false);
+    onSave(ticket);
   };
 
   return (
@@ -3430,8 +3430,8 @@ function TicketModal({ ticket: initTicket, mode, currentUser, currentRole, produ
 
           <div className="modal-actions" style={{ margin: "16px 28px 24px" }}>
             <button type="button" className="btn-cancel" onClick={onClose}>Cancel</button>
-            <button type="submit" className="btn-primary" disabled={saving || !ticket.title.trim()}>
-              {saving ? "Saving…" : mode === "create" ? "Create Ticket" : "Save Changes"}
+            <button type="submit" className="btn-primary" disabled={!ticket.title.trim()}>
+              {mode === "create" ? "Create Ticket" : "Save Changes"}
             </button>
           </div>
         </form>
@@ -3908,15 +3908,15 @@ export default function App() {
   if (!session) return <LoginPage onLogin={handleLogin} />;
   if (view === "review")    return <ReviewPortal    currentUser={session.username} currentRole={session.role} onBack={() => setView(null)} onLogout={handleLogout} onNavigate={handleNavigate} />;
   if (view === "reports")   return <ReportsPortal   currentUser={session.username} currentRole={session.role} onBack={() => setView(null)} onLogout={handleLogout} onNavigate={handleNavigate} />;
-  if (view === "accounts")  return <AccountsPortal  currentUser={session.username} onBack={() => setView(null)} onLogout={handleLogout} />;
-  if (view === "products")  return <ProductsPortal  currentUser={session.username} onBack={() => setView(null)} onLogout={handleLogout} />;
+  if (view === "accounts")  return <AccountsPortal  currentUser={session.username} currentRole={session.role} onBack={() => setView(null)} onLogout={handleLogout} />;
+  if (view === "products")  return <ProductsPortal  currentUser={session.username} currentRole={session.role} onBack={() => setView(null)} onLogout={handleLogout} />;
   if (view === "planner")   return <KanbanPortal    currentUser={session.username} currentRole={session.role} onBack={() => setView(null)} onLogout={handleLogout} onNavigate={handleNavigate} />;
   if (view === "tickets")   return <TicketsPortal   currentUser={session.username} currentRole={session.role} onBack={() => setView(null)} onLogout={handleLogout} onNavigate={handleNavigate} />;
-  if (view === "profiles")  return <TestingProfilesPortal currentUser={session.username}
+  if (view === "profiles")  return <TestingProfilesPortal currentUser={session.username} currentRole={session.role}
     onEdit={p => { setEditingProfile(p); setView("profile-editor"); }}
     onBack={() => setView(null)} onLogout={handleLogout} />;
   if (view === "profile-editor" && editingProfile) return <ProfileEditorPortal
-    profile={editingProfile} currentUser={session.username}
+    profile={editingProfile} currentUser={session.username} currentRole={session.role}
     onBack={() => setView("profiles")} onLogout={handleLogout} />;
   if (session.testingStart && session.testingEnd)
     return <TerminatedScreen session={session} rows={rows}