Skip to content

feat(resources): capacidad, ocupación y fuente oficial del punto - #435

Closed
vgpastor wants to merge 1 commit into
feature/edit-resource-locationfrom
feature/resource-enrichment-fields
Closed

feat(resources): capacidad, ocupación y fuente oficial del punto#435
vgpastor wants to merge 1 commit into
feature/edit-resource-locationfrom
feature/resource-enrichment-fields

Conversation

@vgpastor

Copy link
Copy Markdown
Contributor

Qué

Añade al agregado Resource cuatro campos nuevos (todos nullable), fijables al registrar y editables después vía PATCH /resources/:id, y expuestos en la vista pública:

  • capacity / occupancy — aforo total y ocupación actual del refugio.
  • sourceOrganisation / sourceUrl — organismo oficial que publica el dato y enlace a la fuente (distinto de provenance, que es para ingesta externa automatizada).

Sale del trabajo de carga de los refugios de los incendios: los MD de SITREM traían aforo, ocupación y fuente oficial que hoy acababan como texto libre en description.

Cambios

  • Migración 0059_resource_capacity_source.sql.
  • Dominio (RegisterResourceProps/EditResourceProps/ResourceSnapshot, getters, register/fromSnapshot/edit/toSnapshot).
  • register-resource / edit-resource (comandos + diff de auditoría).
  • DTOs (RegisterResourceDto, EditResourceDto) y vista pública (ResourceView + ResourceViewDto).
  • Repositorio Drizzle: schema, 3 mapeadores, insert y set del upsert.
  • packages/api-client/src/schema.ts regenerado.

Gate

nest build ✅ · eslint/prettier ✅ · 1635 tests ✅ (nuevos tests de edición de capacity/occupancy/fuente y round-trip de snapshot).

Nota

Apilada sobre #434 (base feature/edit-resource-location, comparten ficheros). Al mergear #434, GitHub reapunta esta a main. Servicios y cobertura (municipios/incidentes asignados) quedan fuera de esta tanda por decisión de alcance.

Añade 4 campos nuevos al agregado Resource (nullable), fijables al
registrar y editables después, y expuestos en la vista pública:
- capacity / occupancy (aforo y ocupación del refugio)
- sourceOrganisation / sourceUrl (organismo oficial + enlace de la fuente,
  distinto de `provenance` que es para ingesta externa)

Migración 0059. Vertical completa: dominio (register/edit/snapshot),
DTOs (Register/Edit), vista pública (ResourceView/ResourceViewDto),
repositorio (insert + upsert set + mapeadores) y api-client regenerado.
@vercel

vercel Bot commented Jul 26, 2026

Copy link
Copy Markdown

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Actions Updated (UTC)
response-grid Ready Ready Preview, Comment Jul 26, 2026 4:28am

Request Review

@vgpastor
vgpastor deleted the branch feature/edit-resource-location July 26, 2026 04:40
@vgpastor vgpastor closed this Jul 26, 2026
vgpastor added a commit that referenced this pull request Jul 26, 2026
## Qué

Añade al agregado `Resource` cuatro campos nuevos (todos nullable),
fijables al registrar y editables después vía `PATCH /resources/:id`, y
expuestos en la vista pública:

- **`capacity`** / **`occupancy`** — aforo total y ocupación actual del
refugio.
- **`sourceOrganisation`** / **`sourceUrl`** — organismo oficial que
publica el dato y enlace a la fuente (distinto de `provenance`, ingesta
externa).

Sale de la carga de refugios de los incendios: los MD de SITREM traían
aforo, ocupación y fuente oficial que acababan como texto libre en
`description`.

## Cambios

- Migración `0059_resource_capacity_source.sql`.
- Dominio
(props/snapshot/getters/`register`/`fromSnapshot`/`edit`/`toSnapshot`;
números sin trim, source con trim y vacío→null).
- `register-resource` / `edit-resource` (comandos + diff de auditoría).
- `ingest-external-resources`: preserva los 4 en update, `null` en
insert.
- DTOs (`RegisterResourceDto`, `EditResourceDto`) y vista pública
(`ResourceView` + `ResourceViewDto`).
- Repositorio Drizzle: schema, 3 mapeadores, `insert` y `set` del
upsert.
- `packages/api-client/src/schema.ts` regenerado.

## Gate

`nest build` ✅ · eslint/prettier ✅ · 1635 tests ✅.

## Nota

Reemplaza a #435 (se cerró sola al mergear #434 y borrarse su rama
base). Ya rebasada sobre `main`; el diff es solo el enriquecimiento.
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