Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[17.0][IMP]l10n_es_facturae: readme improve usage #3835

Open
wants to merge 1 commit into
base: 17.0
Choose a base branch
from

Conversation

Yoana-RB
Copy link

@Yoana-RB Yoana-RB commented Dec 9, 2024

He notado que el usage de este addon es algo breve, por lo que he preparado un PR con una mejora que explica de manera más detallada, paso a paso, el proceso y las configuraciones necesarias para su correcto funcionamiento.

@OCA-git-bot
Copy link
Contributor

Hi @pedrobaeza, @etobella,
some modules you are maintaining are being modified, check this out!

@pedrobaeza pedrobaeza added this to the 17.0 milestone Dec 9, 2024
@pedrobaeza
Copy link
Member

Muchas gracias Yoana por mejorar la ayuda del módulo. Varios comentarios para ver si puedes atacarlos:

  1. Cuando esté finalmente todo corregido, debe haber solo un commit que siga las siguientes líneas guía de nomenclatura: https://github.com/OCA/odoo-community.org/blob/master/website/Contribution/CONTRIBUTING.rst#commit-message
  2. Gran parte de lo que has puesto en USAGE.md debe estar en CONFIGURE.md, ya que es parte de la configuración a realizar.
  3. La frase inicial digamos que comercial, podría ir en CONTEXT.md, aunque la idea es dar una idea precisa de lo que hace el módulo, ya que "facturación electrónica" es un término muy amplio.

@Christian-RB Christian-RB force-pushed the 17.0-imp-l10n_es_facturae branch from 471f0e7 to c04c19a Compare December 11, 2024 13:27
@Yoana-RB
Copy link
Author

Hola @pedrobaeza, ya se hicieron los cambios, te agradecería mucho si me pudieras dar algo de feedback al respecto. Muchas gracias por tu apoyo.

Copy link
Member

@pedrobaeza pedrobaeza left a comment

Choose a reason for hiding this comment

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

Gracias por los cambios, Yoana y Christian. Lo único es que se debe mantener trato de usted en la redacción, no de tú. Otra opción es intentar utilizar estilo impersonal.

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.

3 participants