Skip to content

fix(portfolio): contain Campings prototype presentation - #18

Merged
devrodri-com merged 1 commit into
mainfrom
fix/devrodri-dr00-campings-containment
Jul 26, 2026
Merged

fix(portfolio): contain Campings prototype presentation#18
devrodri-com merged 1 commit into
mainfrom
fix/devrodri-dr00-campings-containment

Conversation

@devrodri-com

@devrodri-com devrodri-com commented Jul 25, 2026

Copy link
Copy Markdown
Owner

Contexto y decisión

Campings es un prototipo conceptual e independiente: no fue encargado, adoptado ni afiliado a la organización destinataria. Se conserva únicamente como caso conceptual en el portfolio.

Alcance final de DR-00

  • Mantiene Campings fuera de los destacados de la home y al final de /portfolio.
  • Usa la portada conceptual seleccionada por Rodrigo y generada mediante GPT Images; se presenta como mockup conceptual, no como screenshot exacto de un producto real.
  • Mantiene título, estado y disclaimer aprobados en ES/EN.
  • Mantiene eliminado y prohibido el enlace al deployment externo.
  • Habilita únicamente el repositorio GitHub sanitizado con CTA Ver código en ES y View code en EN.

Gate del repositorio externo

Repositorio: devrodri-com/reservas-campings-nacionales.

  • PR chore(deps): bump react-dom from 18.2.0 to 18.3.1 #16: mergeado en main (3fd977d2ac7da274d5d5ad43e7b2deb71e0ecf3e).
  • README: presentación conceptual e independiente en main, sin enlace al deployment.
  • Descripción pública: neutral y no afiliada.
  • GitHub Homepage: vacía.
  • Enlace GitHub: aprobado.
  • Live demo: continúa prohibida y no se enlaza desde DevRodri.

Riesgo residual: el runtime externo conserva copy/assets institucionales; por eso solo se expone el código sanitizado y no la demo desplegada.

Comportamiento

  • ES: un único CTA externo Ver código hacia https://github.com/devrodri-com/reservas-campings-nacionales.
  • EN: un único CTA externo View code al mismo repositorio.
  • El enlace abre en una pestaña nueva con noopener noreferrer y conserva los estilos de foco existentes.
  • Ver más / View details, filtros, orden, portada, copy y los otros siete casos permanecen intactos.

Validación

  • npm ci: correcto en directorio temporal aislado.
  • ./node_modules/.bin/tsc -b: correcto.
  • npm run build: correcto.
  • git diff --check: correcto.
  • URL GitHub en build: 1 referencia.
  • URL del deployment de Campings en build/runtime: 0 referencias.
  • CTA, títulos, disclaimers y portada aprobada presentes en ES/EN.
  • Diff de esta corrección: un archivo, una propiedad href, sin dependencias ni package files.
  • GitHub Actions CI y Vercel Preview Comments: PASS.
  • Previews devrodri y minisitio-rodrigo: READY y validados en ES/EN a 390, 768, 1024 y 1440 px contra 26b015fdbe5a92d439e100ee626415ab389b4c68.

No objetivos

No modifica el repositorio externo, otros proyectos, home, filtros, estilos, componentes, formulario, Analytics, SEO, rutas, dependencias, CI ni configuración Vercel.

No se realizó merge, push a main ni Production Deployment.

Instrucciones de revisión

  1. Confirmar que Campings no aparece en home y queda último en /portfolio.
  2. Revisar ES/EN: portada, título, estado, disclaimer y CTA Ver código / View code.
  3. Confirmar que el CTA apunta únicamente al repositorio, abre en nueva pestaña y no existe enlace a la demo.
  4. Probar expandir/contraer, filtros y responsive en 390, 768, 1024 y 1440 px.

@vercel

vercel Bot commented Jul 25, 2026

Copy link
Copy Markdown

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

Project Deployment Actions Updated (UTC)
devrodri Ready Ready Preview, Comment Jul 26, 2026 1:18am
minisitio-rodrigo Ready Ready Preview, Comment Jul 26, 2026 1:18am

Request Review

@cursor

cursor Bot commented Jul 26, 2026

Copy link
Copy Markdown

Bugbot is not enabled for your account, so this pull request was not reviewed.

Enable Bugbot in the Cursor dashboard to get automatic reviews on future PRs.

@devrodri-com
devrodri-com merged commit 670f692 into main Jul 26, 2026
4 checks passed
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