Skip to content

Add two-sided station article transfer picker#102

Merged
osswald merged 2 commits into
mainfrom
feature/station-article-transfer-picker
Jul 8, 2026
Merged

Add two-sided station article transfer picker#102
osswald merged 2 commits into
mainfrom
feature/station-article-transfer-picker

Conversation

@osswald

@osswald osswald commented Jul 8, 2026

Copy link
Copy Markdown
Owner

Summary

  • Replace the flat station article multi-select with a two-sided transfer picker: searchable category tree on the left, grouped selected list with filter on the right; click moves articles between sides.
  • Extend minimal article catalog responses with article_category_id / article_category_name so category labels render correctly.
  • Prune layout cell article IDs that are no longer on any station during config autosave to avoid 422 validation errors when rapidly transferring articles.

Test plan

  • Open event config → Stationen and confirm available/selected panels with category labels
  • Add several articles left→right and remove several right→left in quick succession; autosave succeeds without 422
  • Filter both sides and confirm matching results / empty states
  • Confirm layout button articles that are removed from all stations are dropped on save (no stale subset violations)
  • cd cloud/frontend && npm test && npm run typecheck
  • ./scripts/lint.sh

Made with Cursor

Replace the flat multi-select with a categorized available/selected transfer UI, include category fields on minimal article catalog responses, and prune layout cell articles that are no longer on stations during autosave.

Co-authored-by: Cursor <cursoragent@cursor.com>
@cursor

cursor Bot commented Jul 8, 2026

Copy link
Copy Markdown

Bugbot is not enabled for your account, so this pull request was not reviewed.

Enable Bugbot in the Cursor dashboard to get automatic reviews on future PRs.

Harden ArticleMinimalRead so missing category relations do not fail Pydantic validation, and treat null category ids safely when grouping articles in the transfer picker.

Co-authored-by: Cursor <cursoragent@cursor.com>
@osswald osswald merged commit d9716f6 into main Jul 8, 2026
23 checks passed
@osswald osswald deleted the feature/station-article-transfer-picker branch July 8, 2026 09:57
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant