Skip to content

feat(dev): add dev seed data and docker-compose override#31

Draft
aitchdot wants to merge 1 commit into
mainfrom
feat/dev-seed
Draft

feat(dev): add dev seed data and docker-compose override#31
aitchdot wants to merge 1 commit into
mainfrom
feat/dev-seed

Conversation

@aitchdot

@aitchdot aitchdot commented May 11, 2026

Copy link
Copy Markdown
Collaborator

Summary

  • Add `docker-compose.dev.yml` override: starts `seed` service after app healthcheck passes
  • Add `scripts/seed_dev.py`: idempotent seeder with 1 org, 3 locations, 6 categories, 3 users, 5 reports across all statuses, case link, pending deletion request
  • Add `package.json`, `tailwind.config.js`, `scripts/build_css.sh` for local Tailwind CSS rebuild without Docker
  • Add `node_modules/` and `package-lock.json` to `.gitignore`

Usage

```bash
docker compose -f docker-compose.yml -f docker-compose.dev.yml up
```

Seed runs automatically once app is healthy. Users: `superadmin` / `admin` / `casemgr`, password `dev`.

Scope

`docker-compose.dev.yml` · `scripts/seed_dev.py` · `scripts/build_css.sh` · `package.json` · `tailwind.config.js` · `.gitignore`

@aitchdot aitchdot added the dx Developer experience / local tooling label May 11, 2026
@codecov

codecov Bot commented May 11, 2026

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.

📢 Thoughts on this report? Let us know!

@jp1337 jp1337 marked this pull request as ready for review May 13, 2026 10:44
@jp1337 jp1337 marked this pull request as draft May 13, 2026 10:45
@jp1337

jp1337 commented Jul 12, 2026

Copy link
Copy Markdown
Member

Thanks for opening this! 🙏

Heads-up: the branch currently doesn't contain any of the changes described in the summary. feat/dev-seed (head 5619811) has a single empty chore: open PR commit, and both gh pr diff 31 and the Files tab show no changes — none of docker-compose.dev.yml, scripts/seed_dev.py, scripts/build_css.sh, package.json or tailwind.config.js were pushed. Looks like the work was committed locally but never pushed (or the branch got reset).

Could you push the actual changes? A couple of things to keep in mind for when you do, so review goes smoothly:

  • Reuse the existing seeder. app/services/demo_seed.py already seeds an org, locations, admin + case-manager users (with static TOTP), reports across all statuses, messages and case links — gated behind DEMO_MODE=true. Please build on/extend that service rather than reimplementing seeding in a standalone script, so the two don't drift from the models.
  • Keep the dev/dev credentials strictly dev-only. They must never be reachable from the prod compose file or image.
  • Tailwind toolchain would be a new pattern. Today the CSS ships as a committed, hand-maintained app/static/css/site.css — there's no Node/Tailwind build in the repo. Introducing one needs its own justification; happy to discuss the approach first.

Leaving this as a draft — ping me once the diff is up and I'll review properly.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

dx Developer experience / local tooling

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants