Skip to content

fix(web): acceso directo a «Mis puntos» en la home de emergencia (#262) - #270

Merged
vgpastor merged 1 commit into
mainfrom
fix/262-mis-puntos-discoverable
Jul 1, 2026
Merged

fix(web): acceso directo a «Mis puntos» en la home de emergencia (#262)#270
vgpastor merged 1 commit into
mainfrom
fix/262-mis-puntos-discoverable

Conversation

@vgpastor

@vgpastor vgpastor commented Jul 1, 2026

Copy link
Copy Markdown
Contributor

Closes #262

Diagnóstico

El feedback («al crear un punto logístico debe agregarlo a esa emergencia») apuntaba a un fallo de vinculación, pero la vinculación de datos ya funciona: el recurso se crea con ownerUserId y emergencyId (endpoint anidado POST /emergencies/{id}/resources) y GET /emergencies/{id}/resources/mine lo devuelve sin filtrar por verificación.

El problema real (confirmado con el usuario) es de discoverability: tras registrar un punto, la única vía para reencontrarlo dentro de la emergencia era un micro-enlace subrayado de 12.5px enterrado en el pie de la home (EmergencyQuickLinks), invisible en la práctica.

Cambio

Se promueve «Mis puntos en esta emergencia» a una sección de gestión de primer nivel (HelpActionRow) en la home de la emergencia, visible para usuarios autenticados y fuera del gate isActive (la gestión debe estar disponible aunque la emergencia esté en pausa). Enlaza a la página /e/{slug}/mis-puntos ya existente.

Gate

  • pnpm --filter web lint ✅ (solo warning preexistente ajeno)
  • pnpm --filter web build

@vercel

vercel Bot commented Jul 1, 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 1, 2026 11:19pm

Request Review

@vgpastor

vgpastor commented Jul 1, 2026

Copy link
Copy Markdown
Contributor Author

⏸️ No mergear todavía. Este cambio depende de la refactorización en curso de la web pública. Dejar en pausa hasta que esa refactorización aterrice y revisar entonces que el punto de entrada a «Mis puntos» encaja con la nueva estructura.

El recurso ya se crea vinculado a la emergencia (ownerUserId + emergencyId,
endpoint anidado) y GET /resources/mine lo devuelve. El punto de dolor real es
de discoverability: tras registrar un punto, la única vía para reencontrarlo era
un micro-enlace de 12.5px enterrado en el pie de la home.

Se promueve «Mis puntos en esta emergencia» a una sección de gestión de primer
nivel (HelpActionRow) visible para usuarios autenticados, fuera del gate isActive
para que la gestión esté siempre disponible. Enlaza a la página /mis-puntos ya
existente.
@vgpastor
vgpastor force-pushed the fix/262-mis-puntos-discoverable branch from e65fcf5 to 8a2978e Compare July 1, 2026 23:19
@vgpastor

vgpastor commented Jul 1, 2026

Copy link
Copy Markdown
Contributor Author

♻️ Rebasada sobre main (incluye #276/#280 AppBar + #282 + #283).

Al traer main salió a la luz que la refactorización de la web pública (#276) eliminó el enlace a «Mis puntos» del pie y no lo reemplazó → la página /e/{slug}/mis-puntos quedó huérfana (inaccesible desde la UI). Esta PR pasa de "nice-to-have" a necesaria: es el único acceso a esa página.

El rebase, además, habría roto el build: la home nueva ya no define token (ahora el AppBar resuelve la sesión internamente), y mi bloque lo usaba. Corregido importando y definiendo getToken() en la página.

Gate: web lint ✅ · web build ✅. Sin conflictos. Lista para revisar/mergear.

@vgpastor
vgpastor merged commit d82fa55 into main Jul 1, 2026
6 checks passed
@vgpastor
vgpastor deleted the fix/262-mis-puntos-discoverable branch July 1, 2026 23:21
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.

[BUG] El punto logístico creado desde el formulario de una emergencia no queda vinculado a esa emergencia

1 participant