Skip to content

Conversation

@gmz-odoo
Copy link
Contributor

@gmz-odoo gmz-odoo commented Aug 1, 2024

  1. Remove older ‘Introduction section’
  2. Add a table listing and update of the currently supported features and modules with a description of its functionality (like the format/structure displayed for Chile Localization page)
  3. Update the description of suggested and related resources, such as Smart Tutorial videos
  4. Update the reference for current Bank sync options
  5. Update image for current listed Bank sync options
  6. Correct the tittle of the exporting invoicing section

Expected result and comparison of actual info:
https://docs.google.com/document/d/1uyB_2sKIbab26LHO8mx9ikPOUukN6-DjHUz0jkcgXx0/edit

@robodoo
Copy link
Collaborator

robodoo commented Aug 1, 2024

Pull request status dashboard

@gmz-odoo gmz-odoo changed the title [IMP] Update current Peruvian modules and description [IMP] l10n_pe: Update current Peruvian modules and description Aug 1, 2024
@samueljlieber samueljlieber changed the base branch from 17.0 to 16.0 October 15, 2024 19:05
@samueljlieber samueljlieber changed the base branch from 16.0 to 17.0 October 15, 2024 19:06
@C3POdoo C3POdoo requested a review from a team October 15, 2024 19:07
Copy link
Contributor

@samueljlieber samueljlieber left a comment

Choose a reason for hiding this comment

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

Hi @gmz-odoo, thanks for your work on this PR (and on #10563 & #10635)!

These changes look good to me, approving :)

Copy link
Contributor

@dade-odoo dade-odoo left a comment

Choose a reason for hiding this comment

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

Hey again @gmz-odoo thanks for the addition to the Peruvian localization! I've got a few more comments and suggestions here - please let me know if you have any questions on them :)

Remove older ‘Introduction section’
Add a table listing and update of the currently supported features and modules with a description of its functionality (like the format/structure displayed for Chile Localization page)
Update the description of suggested and related resources, such as Smart Tutorial videos
@samueljlieber
Copy link
Contributor

Thank you @dade-odoo and @gmz-odoo! Feedback implemented in 2f966ca!

Copy link
Contributor

@dade-odoo dade-odoo left a comment

Choose a reason for hiding this comment

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

Thanks for making those changes @gmz-odoo and @samueljlieber ! All good for me, I'll pass you on to the final review :)

@dade-odoo dade-odoo requested a review from a team October 24, 2024 12:00
Copy link
Contributor

@xpl-odoo xpl-odoo left a comment

Choose a reason for hiding this comment

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

@robodoo delegate=gmz-odoo,samueljlieber

@samueljlieber
Copy link
Contributor

@robodoo r+

robodoo pushed a commit that referenced this pull request Oct 24, 2024
Remove older ‘Introduction section’
Add a table listing and update of the currently supported features and modules with a description of its functionality (like the format/structure displayed for Chile Localization page)
Update the description of suggested and related resources, such as Smart Tutorial videos

closes #10562

Signed-off-by: Samuel Lieber (sali) <sali@odoo.com>
@robodoo robodoo closed this Oct 24, 2024
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