Skip to content

Remove generated timestamp from exercise README#17

Closed
lav-adhoc wants to merge 2 commits into
mainfrom
claude/dazzling-wright-fXIij
Closed

Remove generated timestamp from exercise README#17
lav-adhoc wants to merge 2 commits into
mainfrom
claude/dazzling-wright-fXIij

Conversation

@lav-adhoc
Copy link
Copy Markdown
Contributor

Summary

Removed the auto-generated timestamp from the billing and payments exercise README file to reduce unnecessary metadata that changes frequently.

Changes

  • Removed the **Generado:** 2026-05-21 line from the README metadata section

Rationale

The generated date field is not essential documentation and creates unnecessary churn in version control when files are regenerated. Keeping only the stable metadata (Module, Odoo Version) improves the signal-to-noise ratio in the repository.

https://claude.ai/code/session_01UCHYjLQJaEJwQcoWVvtMAw

vib-adhoc and others added 2 commits May 21, 2026 22:02
Genera el walkthrough del bloque 03-facturacion-y-pagos a partir del
video, cubriendo los 4 escenarios: carga de facturas de proveedor
(desde OC y manual), pagos a proveedores (banco con retenciones y
caja), facturación a clientes (desde OV y manual) y cobranzas
(transferencia y cheque de tercero).

- Reemplaza el README placeholder por el walkthrough completo de 25
  pasos con capturas reales tomadas en academy-21-05-1.adhoc.inc.
- Marca el bloque 03 como `listo` en el índice de ejercicios.
Copilot AI review requested due to automatic review settings May 22, 2026 16:10
Copy link
Copy Markdown

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

Removes the auto-generated “Generado” timestamp line from the billing/payments exercise README to reduce noisy, frequently changing metadata in version control.

Changes:

  • Deleted the **Generado:** 2026-05-21 metadata line from the README header.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Base automatically changed from video-facturacion-y-pagos to main May 28, 2026 16:55
jjscarafia added a commit that referenced this pull request Jun 2, 2026
Unifica la mezcla "Nativa Skill" / "Natura Skin" hacia el nombre canónico
del caso (Natura Skin, definido en contexto.md, confirmado contra la
presentación fuente). Cambios:

- Reemplaza "Nativa Skill(s)" -> "Natura Skin" y el login
  sofia@nativa.com -> sofia@natura.com en los artefactos de procesos
  (result-v2, result-0805: transcript, steps.json, scripts de captura).
- Agrega el login canónico (sofia@natura.com) a la tabla de datos de
  empresa en contexto.md como fuente única.
- Nota de transcripción en transcript.txt aclarando que el audio dice
  "Nativa Skill" pero se unificó al nombre canónico.
- Quita la línea "Generado:" del README del bloque 03 (churn innecesario;
  era el único aporte del PR #17, ahora obsoleto).

Nota operativa: los scripts de captura usan el login como credencial
real contra la base Odoo de test — la base debe tener el usuario
sofia@natura.com para que la automatización siga funcionando.

Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
@jjscarafia
Copy link
Copy Markdown
Contributor

Cierro este PR: quedó obsoleto.

  • Su único aporte real era quitar la línea **Generado:** del README de bloque 03, lo cual ya se aplicó directamente en main (commit 9ea172e).
  • La branch está basada en una versión de main anterior a la reorganización de carpetas de ejercicios, así que mergearlo hoy revertiría esa reestructuración (recrearía 01-setup-empresa.md, borraría 04-localizaciones, desharía renames de 02-compras-y-ventas). Sería destructivo.

No hay nada más que rescatar. Cerrado sin merge.

@jjscarafia jjscarafia closed this Jun 2, 2026
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.

5 participants