Skip to content

Novedades: un aviso de cambios abre con el cambio primero - #329

Merged
alejooroncoy merged 1 commit into
mainfrom
codex/campu-aviso-de-cambios
Nov 11, 2026
Merged

alejooroncoy merged 1 commit into
mainfrom
codex/campu-aviso-de-cambios

Conversation

@alejooroncoy

Copy link
Copy Markdown
Owner

Problema

El correo de cambios ("Encontré cambios importantes en tu campus") lleva a /app/novedades/<id>/. Con cambios concretos, la página mostraba el detalle completo y el cambio quedaba en el quinto bloque, debajo de todos los ajustes. Se ve en producción con una cuenta real: se toca "ver" y aparece configuración (reproducido en el navegador integrado: aviso de tipo changes, 1 evento feedback_changed de S07|Trabajo Parcial).

Cambio

  • La ruta del aviso abre la vista simple también cuando hay cambios (antes lo excluía).
  • Titular en una frase: "Cambió la retroalimentación de X en Y.", "Publicaron una nota de…", "Encontré N cambios en tus cursos."
  • La tarjeta "Cambios de este aviso" va primero, antes de "Esta semana". Se extrajo a ChangesCard, usada en la vista simple y en el detalle, sin duplicarse al abrirlo.
  • Ajustes y detalle siguen detrás del botón.

Pruebas

npm run check sin errores y proactive-e2e en verde a 390 y 1100 px (con Chrome del sistema, porque falta el navegador de Playwright en el worktree). Nuevas aserciones: el titular del cambio, que no haya ajustes antes de abrir nada, que el cambio se vea sin abrir el detalle y que vaya antes que lo que viene. Se ajustó una sección que abría el detalle antes de que la vista cambiara a la simple tras reintentar.

🤖 Generated with Claude Code

El correo de cambios lleva a /app/novedades/<id>/. Con cambios concretos la página
mostraba el detalle completo, y el cambio quedaba en el quinto bloque, debajo de
todos los ajustes: la persona tocaba ver y encontraba configuración.

Ahora esa ruta también abre la vista simple, con el cambio en una frase
(Cambió la retroalimentación de X en Y) y su tarjeta arriba, antes de lo que viene.
Los ajustes y el detalle siguen detrás de Ajustes y detalle, y el cambio no se
duplica al abrirlos.

Co-Authored-By: Claude Sonnet 5.5 <noreply@anthropic.com>
@vercel

vercel Bot commented Nov 10, 2026 •

Copy link
Copy Markdown

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

Project Deployment Actions Updated
landing Ready Ready Preview Nov 10, 2026 6:14pm UTC

@alejooroncoy

Copy link
Copy Markdown
Owner Author

@codex review

@chatgpt-codex-connector

chatgpt-codex-connector Bot commented Nov 10, 2026 •

Copy link
Copy Markdown
Codex Review Summary: ❌ Failed

This comment shows the latest Codex review activity on this pull request.

Review Status Commit Review trigger
📝 Code Review ❌ Failed 2026-11-10T18:14:23.429393Z
We couldn’t complete this review. Comment @codex review to try again.
375d4f1 Manual request
ℹ️ About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

  • Open a pull request for review
  • Mark a draft as ready
  • Comment "@codex review" or "@codex security review".

Codex reacts with 👀 while any review is running, comments if it has suggestions, and reacts with 👍 once all reviews finish with no findings.

@chatgpt-codex-connector

Copy link
Copy Markdown

You have reached your Codex usage limits for code reviews. You can see your limits in the Codex usage dashboard.
To continue using code reviews, you can upgrade your account or add credits to your account and enable them for code reviews in your settings.

@chatgpt-codex-connector

Copy link
Copy Markdown

You have reached your Codex usage limits for code reviews. You can see your limits in the Codex usage dashboard.
To continue using code reviews, you can upgrade your account or add credits to your account and enable them for code reviews in your settings.

@alejooroncoy
alejooroncoy merged commit 86c2c49 into main Nov 11, 2026
3 checks passed

This branch was successfully deployed

1 active deployment
Preview — 375d4f10 Deployed Nov 10, 2026 by vercel[bot]
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