Skip to content

Webhooks: docs/webhooks-info → main - #2

Merged
mendozagit merged 2 commits into
mainfrom
docs/webhooks-info
Oct 14, 2026
Merged

mendozagit merged 2 commits into
mainfrom
docs/webhooks-info

Conversation

@mendozagit

@mendozagit mendozagit commented Oct 14, 2026 •

Copy link
Copy Markdown
Contributor

Resumen

README y docs del CLI enlazan a /webhooks-info y muestran la salida real de los comandos.

Commits

  • docs: la guía de webhooks pasa a docs.fiscalapi.com/webhooks-info
  • docs: salida real de --json, events retrieve, Actualizado y 400 de validación

Summary by CodeRabbit

  • Documentación
    • Actualizados los enlaces a la guía de webhooks.
    • Aclarado cómo se muestran las respuestas de events retrieve y --json, y que la salida de eventos no sirve para verificar firmas.
    • Añadidos ejemplos del campo Actualizado en las operaciones de webhooks.
    • Ampliada la guía de errores 400 y 401, con detalles sobre las causas y la presentación de errores de validación.

fapi-docs separa la guía (tutorial /webhooks-info) de la referencia de endpoints (/webhooks y /events).
El README y listen.md enlazan a la guía: evento, firma y reintentos.
…lidación

Diferencias que encontró un E2E del CLI 0.1.0 contra el back:
- --json imprime el data con GetRawText (ApiCommandContext.cs:34), es decir con el formato indentado de
  la API (WriteIndented en el back). Los ejemplos de una línea pasan a ese formato.
- events retrieve muestra el evento como lo devuelve la API, no el cuerpo compacto que se entregó; se
  aclara que no sirve para verificar una firma.
- webhooks create y retrieve siempre imprimen Actualizado (WebhookView.WriteDetail; al crear es igual a Creado).
- Un 400 de validación pone el texto general en Detalle y una línea "- <mensaje>" por regla (ApiException.cs:73-76).
- solucion-de-problemas: se separan el 401 sin header X-API-KEY y el 401 Invalid API KEY.
@coderabbitai

coderabbitai Bot commented Oct 14, 2026

Copy link
Copy Markdown

Review in Change Stack →

Note

Currently processing new changes in this PR. This may take a few minutes, please wait...

⚙️ Run configuration
  • Configuration used: defaults
  • Review profile: CHILL
  • Plan: Advanced
  • Run ID: ce5a8f8c-94f1-49cd-8203-a6f7bd6fa238
📥 Commits

Reviewing files that changed from the base of the PR and between 46eacd4 and cf81f81.

📒 Files selected for processing: 6
  • README.md
  • docs/listen.md
  • docs/salida-y-codigos.md
  • docs/solucion-de-problemas.md
  • docs/trigger.md
  • docs/webhooks-y-eventos.md
 _________________________________
< My bunny whiskers are tingling. >
 ---------------------------------
  \
   \   \
        \ /\
        ( )
      .( o ).
✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Commit to this branch
  • Create a new PR
  • Autofix · Keep fixing CodeRabbit findings and required CI, and resolving merge conflicts

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands.

@mendozagit
mendozagit merged commit e487a47 into main Oct 14, 2026
1 of 2 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