Skip to content

fix(validatie): bulk-seed is default in bootstrap (~1000 studenten)#60

Merged
EdF2021 merged 1 commit intomainfrom
fix/bootstrap-default-bulk-seed
May 9, 2026
Merged

fix(validatie): bulk-seed is default in bootstrap (~1000 studenten)#60
EdF2021 merged 1 commit intomainfrom
fix/bootstrap-default-bulk-seed

Conversation

@EdF2021
Copy link
Copy Markdown
Contributor

@EdF2021 EdF2021 commented May 9, 2026

Probleem

bootstrap.sh en de beheerpagina gebruikten seed.py (3 studenten + 2 mentoren, dev-demo) als default. seed_bulk.py (~1000 studenten over alle 5 instellingen, deterministisch met RNG=2026) zat achter een opt-in flag. Werkpraktijk is precies omgekeerd: 1000 studenten is de baseline.

Wijzigingen

  • scripts/bootstrap.sh: default = seed_bulk.py. --seed-bulk flag vervangen door --seed-minimal (opt-in 3+2 voor handmatige UI-tests). --skip-seed blijft.
  • app/pages/9_beheer.py Bootstrap-tab: radio i.p.v. drie checkboxes; "Bulk (~1000 studenten over alle OERs) — default" staat bovenaan en is initieel geselecteerd.
  • app/pages/9_beheer.py Seed-tab: bulk-knop is type="primary" en boven; seed.py-knop secondair met expliciete "dev-demo" labeling.
  • CLAUDE.md: commando-tabel en beheerpagina-uitleg bijgewerkt.

Test plan

  • uv run python -m pytest (63/63 groen)
  • uv run ruff check src/ app/ scripts/
  • bash -n scripts/bootstrap.sh
  • Reviewer: open /beheer → Bootstrap-tab → bevestig dat radio default op "Bulk" staat
  • Reviewer: dry-run ./scripts/bootstrap.sh --skip-sync op een dev-machine om te zien dat seed_bulk.py draait i.p.v. seed.py

🤖 Generated with Claude Code

Daisy werkt met ~1000 studenten als baseline. `bootstrap.sh` riep voorheen
`seed.py` (3 studenten + 2 mentoren, dev-demo) standaard aan en `seed_bulk.py`
zat achter een opt-in `--seed-bulk`-flag — precies omgekeerd van wat de
werkpraktijk vraagt.

- `bootstrap.sh`: default = `seed_bulk.py`. Vervangen `--seed-bulk` door
  `--seed-minimal` (opt-in 3+2 voor handmatige UI-tests). `--skip-seed`
  blijft beschikbaar.
- `9_beheer.py` Bootstrap-tab: radio-keuze met "Bulk (default)", "Minimaal",
  "Geen" — bulk staat bovenaan en is initieel geselecteerd.
- `9_beheer.py` Seed-tab: bulk-knop is `type="primary"` en boven; seed.py is
  secondair met expliciete "dev-demo" labeling.
- CLAUDE.md werkt de commando-tabel en beheerpagina-uitleg bij.

Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
@EdF2021 EdF2021 merged commit e74d357 into main May 9, 2026
1 check passed
@EdF2021 EdF2021 deleted the fix/bootstrap-default-bulk-seed branch May 9, 2026 11:49
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