feat(web): enlazar el Tecnoboletín desde la navegación y la portada - #49
Merged
Conversation
El boletin llevaba tiempo publicado en /tecnoboletin pero no habia forma de llegar desde aqui: cero enlaces y cero menciones en toda la home. Solo se accedia sabiendose la URL. - Entrada "Boletin" en NAV, entre Articulos y Talleres. Al venir de `data/site.ts` aparece en la cabecera de las 92 paginas del sitio. - Seccion propia en la portada, entre "Ultimos articulos" y "Talleres", con el mismo patron que el resto (eyebrow + h2 + entradilla + grid-2 de tarjetas): una tarjeta al boletin del dia y otra al grafo. La tarjeta principal apunta a /tecnoboletin/hoy/ y no a una fecha concreta: es la URL estable que siempre sirve la ultima edicion. La entrada de navegacion apunta a la raiz de la seccion. /tecnoboletin es otro despliegue bajo el mismo dominio, no una ruta de este sitio Astro, asi que son enlaces normales y no entran en el sitemap de aqui. Queda anotado en el comentario de NAV para que no sorprenda. Verificado sobre el build: la portada tiene los 3 enlaces (nav + 2 tarjetas) y el de navegacion sale en las 92 paginas construidas. Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
There was a problem hiding this comment.
Pull request overview
Este PR hace visible el Tecnoboletín dentro del sitio principal, añadiendo enlaces desde la navegación global y desde la portada para que la sección /tecnoboletin deje de ser “solo accesible por URL”.
Changes:
- Añade una entrada “Boletín” en la navegación (NAV) apuntando a
/tecnoboletin/, con nota aclaratoria de que es un despliegue externo al Astro del sitio. - Incorpora una nueva sección “Tecnoboletín” en la portada con dos tarjetas: “Boletín de hoy” (
/tecnoboletin/hoy/) y “Grafo de conocimiento” (/tecnoboletin/grafo/).
Reviewed changes
Copilot reviewed 2 out of 2 changed files in this pull request and generated 1 comment.
| File | Description |
|---|---|
| src/pages/index.astro | Añade la sección de portada para enlazar el Tecnoboletín (hoy + grafo). |
| src/data/site.ts | Agrega el enlace “Boletín” a la navegación global y documenta que es un despliegue externo. |
Suppressed comments (1)
src/pages/index.astro:162
- Tras insertar la sección de Tecnoboletín, la sección real de Talleres ya no tiene su comentario/etiqueta de bloque. Si estos comentarios se usan como separadores visuales, conviene reintroducir
<!-- talleres -->justo encima de la sección de Talleres.
</section>
<section class="section" style="padding-top: 0;">
💡 Add a code-review agent skill or configure MCP servers for context-aware, tailored reviews. Learn more in the docs.
Comment on lines
140
to
+141
| <!-- talleres --> | ||
| <!-- tecnoboletín --> |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
El Tecnoboletín lleva tiempo publicado en
/tecnoboletinpero no había forma de llegar desde el sitio: cero enlaces y cero menciones de «boletín» en toda la portada. Solo se accedía sabiéndose la URL.Qué cambia
data/site.ts, aparece en la cabecera de las 92 páginas del sitio.h2+ entradilla +grid-2de tarjetas): una tarjeta al boletín del día y otra al grafo de conocimiento.Decisiones
/tecnoboletin/hoy/, no a una fecha concreta: es la URL estable que siempre sirve la última edición. La entrada de navegación apunta a la raíz de la sección./tecnoboletines otro despliegue bajo el mismo dominio, no una ruta de este sitio Astro. Por eso son enlaces normales y no entran en el sitemap de aquí. Queda anotado en un comentario junto aNAVpara que no sorprenda.Verificación
pnpm buildcompleto. Sobre el resultado:/tecnoboletin/(nav),/tecnoboletin/hoy/y/tecnoboletin/grafo/(tarjetas).🤖 Generated with Claude Code