Integrate playground-approved kanban into pal-e-app #47

New issue

Open

opened 2026-03-23 14:16:18 +00:00 by forgejo_admin · 2 comments

forgejo_admin commented

2026-03-23 14:16:18 +00:00

Contributor

Copy link

Type

Feature

Lineage

Standalone — board-pal-e-docs kanban MVP

Repo

forgejo_admin/pal-e-app

User Story

As the superuser
I want the playground-approved kanban design in pal-e-app
So that visiting pal-e-app shows me the kanban I designed

Context

Depends on playground kanban prototype being approved by Lucas. The playground HTML/CSS gets promoted into SvelteKit — .svelte files ARE HTML, so this is copy-paste + data bindings to the real API.

File Targets

Files to modify:

src/routes/boards/[slug]/+page.svelte — replace with playground-approved design
src/app.css — merge any new CSS vars from playground

Files NOT to touch:

src/routes/boards/[slug]/+page.server.ts — data loading stays the same
API proxy routes — already work

Acceptance Criteria

Board view matches playground-approved design
Real API data renders correctly
Drag-and-drop works with optimistic updates
Auth gating: board mutations require login

Test Expectations

Visual comparison: production matches playground
Drag item between columns, verify API call succeeds
Unauthenticated user can view but not modify

Constraints

No Tailwind — pure CSS
.svelte files ARE HTML — copy-paste from playground + add Svelte data bindings
Don't break existing board functionality (create/delete items)

Checklist

PR opened
Tests pass
No unrelated changes

project-pal-e-docs — pal-e-app is the frontend

### Type Feature ### Lineage Standalone — board-pal-e-docs kanban MVP ### Repo `forgejo_admin/pal-e-app` ### User Story As the superuser I want the playground-approved kanban design in pal-e-app So that visiting pal-e-app shows me the kanban I designed ### Context Depends on playground kanban prototype being approved by Lucas. The playground HTML/CSS gets promoted into SvelteKit — .svelte files ARE HTML, so this is copy-paste + data bindings to the real API. ### File Targets Files to modify: - `src/routes/boards/[slug]/+page.svelte` — replace with playground-approved design - `src/app.css` — merge any new CSS vars from playground Files NOT to touch: - `src/routes/boards/[slug]/+page.server.ts` — data loading stays the same - API proxy routes — already work ### Acceptance Criteria - [ ] Board view matches playground-approved design - [ ] Real API data renders correctly - [ ] Drag-and-drop works with optimistic updates - [ ] Auth gating: board mutations require login ### Test Expectations - [ ] Visual comparison: production matches playground - [ ] Drag item between columns, verify API call succeeds - [ ] Unauthenticated user can view but not modify ### Constraints - No Tailwind — pure CSS - .svelte files ARE HTML — copy-paste from playground + add Svelte data bindings - Don't break existing board functionality (create/delete items) ### Checklist - [ ] PR opened - [ ] Tests pass - [ ] No unrelated changes ### Related - `project-pal-e-docs` — pal-e-app is the frontend

forgejo_admin commented

2026-03-28 05:42:38 +00:00

Author

Contributor

Copy link

Scope Review: NEEDS_REFINEMENT

Review note: review-298-2026-03-27
Ticket is well-structured but has fixable issues before moving to next_up.

[LABEL] Missing arch:frontend label on board item #298
[BODY] Stale file target: +page.server.ts no longer exists after adapter-static migration — remove from "Files NOT to touch"
[BODY] Add explicit dependency note: board item #297 (playground kanban prototype, issue #46) must be done first
[SCOPE] Clarify: full rewrite of the 31k +page.svelte or incremental changes on top of the existing ported board page (#72)?

## Scope Review: NEEDS_REFINEMENT Review note: `review-298-2026-03-27` Ticket is well-structured but has fixable issues before moving to next_up. - **[LABEL]** Missing `arch:frontend` label on board item #298 - **[BODY]** Stale file target: `+page.server.ts` no longer exists after adapter-static migration — remove from "Files NOT to touch" - **[BODY]** Add explicit dependency note: board item #297 (playground kanban prototype, issue #46) must be done first - **[SCOPE]** Clarify: full rewrite of the 31k +page.svelte or incremental changes on top of the existing ported board page (#72)?

forgejo_admin commented

2026-03-28 18:39:13 +00:00

Author

Contributor

Copy link

Scope Review: NEEDS_REFINEMENT

Review note: review-298-2026-03-28

Ticket has 6 issues that need fixing before it can move to next_up.

Dependency blocker: Board item #297 (playground kanban prototype, issue #46) is still in_progress. This ticket's Context says "Depends on playground kanban prototype being approved by Lucas." Cannot proceed until #297 is done.
Stale repo reference: Issue body says forgejo_admin/pal-e-app but repo was renamed to forgejo_admin/pal-e-docs-app
Stale file target: +page.server.ts "do not touch" reference — file does not exist (app uses adapter-static since PR #57)
Missing traceability: No arch:frontend label on board item
Misleading AC: AC 2/3/4 (API data, drag-drop, auth gating) are already implemented in the existing 1195-line board page. Only AC 1 (visual restyling) describes new work. AC should be rewritten to reflect the actual delta.
Scope clarity needed: "Copy-paste from playground + data bindings" mischaracterizes the work. Playground is 132 lines of static HTML; the app board page is 1195 lines of working Svelte with drag-drop, filters, and optimistic updates. Real task is CSS restyling, not a rewrite.

## Scope Review: NEEDS_REFINEMENT Review note: `review-298-2026-03-28` Ticket has 6 issues that need fixing before it can move to next_up. - **Dependency blocker**: Board item #297 (playground kanban prototype, issue #46) is still `in_progress`. This ticket's Context says "Depends on playground kanban prototype being approved by Lucas." Cannot proceed until #297 is done. - **Stale repo reference**: Issue body says `forgejo_admin/pal-e-app` but repo was renamed to `forgejo_admin/pal-e-docs-app` - **Stale file target**: `+page.server.ts` "do not touch" reference — file does not exist (app uses adapter-static since PR #57) - **Missing traceability**: No `arch:frontend` label on board item - **Misleading AC**: AC 2/3/4 (API data, drag-drop, auth gating) are already implemented in the existing 1195-line board page. Only AC 1 (visual restyling) describes new work. AC should be rewritten to reflect the actual delta. - **Scope clarity needed**: "Copy-paste from playground + data bindings" mischaracterizes the work. Playground is 132 lines of static HTML; the app board page is 1195 lines of working Svelte with drag-drop, filters, and optimistic updates. Real task is CSS restyling, not a rewrite.

Commenting is not possible because the repository is archived.

No Branch/Tag specified

main

110-revert-rename

107-feat-extractboardcontext-slug-extraction

104-board-html-content

226-fix-wget-auth

No results found.

Labels

Clear labels

No labels

Milestone

Clear milestone

No items

No milestone

Projects

Clear projects

No items

No project

Assignees

Clear assignees

No assignees

1 participant

Due date

The due date is invalid or out of range. Please use the format "yyyy-mm-dd".

No due date set.

Dependencies

No dependencies set.

Reference

ldraney/pal-e-app#47

Reference in a new issue

Repository

ldraney/pal-e-app

Title

Body

No description provided.

Delete branch "%!s()"

Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?

Rows
Columns