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

[IMP] l10n_pe: Clean the manifest, remove to-dos #129186

Closed
wants to merge 1 commit into from

Conversation

fvz-odoo
Copy link
Contributor

@fvz-odoo fvz-odoo commented Jul 20, 2023

At present, the manifest for the base module in Peru contains two pending tasks that need further clarification. These to-dos have remained unresolved since the module's inception four years ago. However, with this recent commit, we have provided a clearer explanation of what the module offers and have addressed and resolved the pending to-dos.

Task id=3337432

@robodoo
Copy link
Contributor

robodoo commented Jul 20, 2023

@C3POdoo C3POdoo requested review from a team and Levizar and removed request for a team July 20, 2023 22:47
@C3POdoo C3POdoo added CLA Contributor License Agreement Localization about adapting the content to a specify country (usually chart of accounts) labels Jul 20, 2023
@ren-odoo
Copy link
Contributor

@fvz-odoo the name of the commit must follow this format: https://www.odoo.com/documentation/16.0/contributing/development/git_guidelines.html#commit-message-structure, can you take a look and update it pls.

Thank you.

@fvz-odoo
Copy link
Contributor Author

@ren-odoo done!

@fvz-odoo fvz-odoo changed the title [IMP] l10n_pe: Clean the manifest (remove to-dos) [IMP] l10n_pe: Clean the manifest, remove to-dos Jul 31, 2023
When l10n_pe was initially developed, there were two unresolved matters that needed clarification for the users. However, with this recent commit, the manifest has been cleaned up, and now the explanation of the module is much clearer and easily understandable for the users.”
@Levizar
Copy link
Contributor

Levizar commented Feb 23, 2024

Should I close this pr?

@fvz-odoo
Copy link
Contributor Author

Hello @Levizar

How about merging it? Or should I do something differently?

Cheers!

Fernando

Copy link
Contributor

@Levizar Levizar left a comment

Choose a reason for hiding this comment

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

You're right, I didn't take the time to check if it was still like that in master but it seems that it is.

@robodoo r+

robodoo pushed a commit that referenced this pull request Feb 23, 2024
When l10n_pe was initially developed, there were two unresolved matters that needed clarification for the users. However, with this recent commit, the manifest has been cleaned up, and now the explanation of the module is much clearer and easily understandable for the users.”

closes #129186

Signed-off-by: Brice Bartoletti (bib) <bib@odoo.com>
@robodoo robodoo closed this Feb 23, 2024
gamarino pushed a commit to numaes/numa-public-odoo that referenced this pull request Mar 1, 2024
When l10n_pe was initially developed, there were two unresolved matters that needed clarification for the users. However, with this recent commit, the manifest has been cleaned up, and now the explanation of the module is much clearer and easily understandable for the users.”

closes odoo/odoo#129186

Signed-off-by: Brice Bartoletti (bib) <bib@odoo.com>
kaynnan pushed a commit to Escodoo/OCB that referenced this pull request Mar 25, 2024
When l10n_pe was initially developed, there were two unresolved matters that needed clarification for the users. However, with this recent commit, the manifest has been cleaned up, and now the explanation of the module is much clearer and easily understandable for the users.”

closes odoo#129186

Signed-off-by: Brice Bartoletti (bib) <bib@odoo.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
CLA Contributor License Agreement Localization about adapting the content to a specify country (usually chart of accounts)
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

5 participants