Node page: missing/denied node spins 'Loading node…' forever; 'No calls recorded' is wrong for quiet nodes #288

New Issue

2026-10-25T21:32:14-04:00

logan commented

2026-10-25 21:32:14 -04:00

Found in a UX pass 2026-10-25. Related: #109 item 5. Searched open+closed: no duplicate.

/nodes/[id] shows "Loading node…" forever when the doc does not exist, was just deleted by another admin, or the read is denied: onSnapshot (app/nodes/[id]/page.tsx:51-53) maps a missing doc to null, has no error callback, and if (!node) (:127) renders the loading text for both. Should distinguish loading / not found / no access, with a link back to /nodes.
"No calls recorded from this node." (:~254) is computed from useCalls(200) filtered client-side (:44-48): a quiet node in a busy org shows "none" although it has calls. Needs the server-side node_id filter (index) -- the TODO at :41.

Found in a UX pass 2026-10-25. Related: #109 item 5. Searched open+closed: no duplicate. - `/nodes/[id]` shows "Loading node…" forever when the doc does not exist, was just deleted by another admin, or the read is denied: `onSnapshot` (`app/nodes/[id]/page.tsx:51-53`) maps a missing doc to `null`, has no error callback, and `if (!node)` (`:127`) renders the loading text for both. Should distinguish loading / not found / no access, with a link back to /nodes. - "No calls recorded from this node." (`:~254`) is computed from `useCalls(200)` filtered client-side (`:44-48`): a quiet node in a busy org shows "none" although it has calls. Needs the server-side node_id filter (index) -- the TODO at `:41`.

logan added the

labels 2026-10-25 21:32:14 -04:00

logan referenced this issue

2026-10-25 21:32:22 -04:00

Frontend punch-list (drb.cusano.net live review, 2026-09-07) #109

Sign in to join this conversation.