fix: Knowledge page uses window.location.reload() for list refresh #27

New issue

Closed

opened 2026-04-09 21:22:08 +00:00 by KnallerKalle · 1 comment

KnallerKalle commented

2026-04-09 21:22:08 +00:00

(Migrated from github.com)

Problem

The knowledge page (src/app/[locale]/knowledge/page.tsx) uses window.location.reload() in handleSaveComplete() after creating/editing a knowledge entry. This causes:

Full page reload (bad UX, loses scroll position)
Database re-initialization on every save
Flash of white screen
Lost application state

Solution

Replace with proper React state management:

Add a refreshKey state counter
Increment on save to trigger KnowledgeList re-render
Pass refreshKey as prop or use a callback to reload list data

Files Affected

src/app/[locale]/knowledge/page.tsx - Replace reload with state update
src/components/knowledge/KnowledgeList.tsx - Accept refresh trigger prop

## Problem The knowledge page (`src/app/[locale]/knowledge/page.tsx`) uses `window.location.reload()` in `handleSaveComplete()` after creating/editing a knowledge entry. This causes: 1. Full page reload (bad UX, loses scroll position) 2. Database re-initialization on every save 3. Flash of white screen 4. Lost application state ## Solution Replace with proper React state management: - Add a `refreshKey` state counter - Increment on save to trigger KnowledgeList re-render - Pass refreshKey as prop or use a callback to reload list data ## Files Affected - `src/app/[locale]/knowledge/page.tsx` - Replace reload with state update - `src/components/knowledge/KnowledgeList.tsx` - Accept refresh trigger prop

KnallerKalle commented

2026-04-09 21:39:43 +00:00

(Migrated from github.com)

Fixed in commit 36f3b12

Replaced window.location.reload() with refreshKey state counter
Added refreshKey prop to KnowledgeList component
List now refreshes via React state instead of full page reload
Build passes, 27/27 tests passing

Fixed in commit 36f3b12 - Replaced window.location.reload() with refreshKey state counter - Added refreshKey prop to KnowledgeList component - List now refreshes via React state instead of full page reload - Build passes, 27/27 tests passing