[ADD] l10n: Ecuador fiscal doc

[samueljlieber]

Task: #3164933

Targets: 16.0

This PR adds Ecuador fiscal localization documentation to Odoo user docs.

[robodoo]

Pull request status dashboard.

[samueljlieber]

Hi @vbe-odoo 👋 I've created the PR for the l10n EC doc! Thank you for putting together the RST in the google doc 🙂

Throughout the doc I made adjustments to the RST and document layout, the changes are summarized below along with my reasoning:

• Moved the Module installation under Configuration heading and removed the steps to install l10n_ec modules since they are covered by the Install reference on line 23 - similar reasoning in [IMP] l10n: update AR documentation #2430 (comment)
• I did not include the screenshots for the Advanced Settings tab for Customer Invoice, Withholding, & Purchase Liquidations because I felt the notes for these steps were descriptive enough, and it helps keep the image count down.
• Minor adjustments to sentence structure.

Overall this is a really great start and the doc ready for your review now that the RST & PR is created. Please let me know your thoughts on the changes 🙂

[vbe-odoo]

Hello @samueljlieber
Many thanks for creating the PR and improving the EC documentation shared with you.

I've done a review, and I agree with your changes!
It looks good to me!

I've moved this PR to Open by mistake, if you want you can move it back to draft.

IMPORTANT NOTE: But it is good to clarify that the EC localization is not yet released - we are close to it - so if the review keeps moving to another team, please wait for our good-to-go before merging this into the User Doc.

Thanks!

[vbe-odoo]

I agree with your changes and the Documentation Looks Good to me.
cc' @ren-odoo @fvz-odoo

[StraubCreative]

@vbe-odoo moved back to draft.
While we're waiting for EC release, I can review it before we open the PR and pass to accounting team for merge 👍

[vbe-odoo]

@StraubCreative we've just r+ the modules of the EC localization 🥳
So this is ready to be moved to the accounting team when ready from your side.
Thanks!
cc' @ren-odoo @fvz-odoo

[StraubCreative]

@samueljlieber there's a number of merge conflicts here.
Can you prioritize solving them and publish the PR asap so accounting team can review?
Then work with accounting and VBE on change requests until merge.
TIA/KR 🙏

[samueljlieber]

Updates in d81207b:

• rebased to latest 16.0 updates (finance directory restructure) to resolve merge conflicts
• fixed build errors

Thank you @StraubCreative & @vbe-odoo for reviewing the changes! Sending to @odoo/accounting-doc-review!

[ren-odoo]

Hi @odoo/accounting-doc-review

Can you take a look at this PR please? We're waiting on this documanation to be published in order to share the oficial release of the Ecuatorian Localization.

Thank you.

[Donapi]

Hello, thank you for this new page, great work :)
I did a high-level review. Here are some general comments based on what we do from our side; feel free to consider them or not:

• add more links to existing documentation
• avoid putting too many images, especially screenshots of whole screens. We prefer to add screenshots that focus on parts of the screen that illustrate a specific point of what is described to make it easier to understand.
• we prefer to write in the present tense than using future.

[Donapi]

I wouldn't add a dedicated Glossary paragraph but rather a short introduction explaining what is SRI

EDI --> explain it in the Electronic Documents section
RIMPE --> what is the abbreviation of? I would explain it in the text when you mention it.

[ren-odoo]

Hi @Donapi for this section, we have recieved feedback from end user and internal staff that's good and practival to have the glosary from the start, that's why we added here. I just did some adjustments for the abreviations.

[samueljlieber]

@Donapi thank you for the suggestions!

Updates made in 133313c:

• added links to documentation for EDI, chart of accounts, and credit notes
• removed an unnecessary chart of accounts image
• squashed @ren-odoo's revisions into one commit

@ren-odoo thank you for making some of the edits, can you help me with these remaining? 🙏

1. Short introduction instead of a glossary
2. Explain where the user has to navigate to for Withholding configuration
3. Explain where the user has to navigate to for Purchase Liquidation configuration
4. Explain where the user has to navigate to for configuring withholding code
5. Confirm report path and take new screenshot that shows where to find both reports

Thank you so much!

[ren-odoo]

Hi @Donapi

All the suggestions and changes requested changes have been addressed.

Is it possible release this first version and then if additional improvements are dectected we handle them in a new PR.

Our EC market and the Sales teams are already waiting for the Content, so it will be ideal if we can release it early next week.

Thank @samueljlieber for the additional changes.

[StraubCreative]

Squashed commits after change requests round + rebased on fade918.

Markup cleaning on e83dce2. Checks pass 👍

[ren-odoo]

Hi @Donapi
Can you help us to check the last changes on this PR please?

We expect to do the offical release of the EC localziation this week, but we depend on this PR to be merged.

Thank you so much.

[samueljlieber]

0cf7078 Updates:

• squashed and rebased @ren-odoo commits

Ready for review 🙂

[vbe-odoo]

Hi @Donapi,
I hope you are doing well!

All changes were done and squashed, can you help us to review this PR?
As it would be ideal to have it for this week so we can do the official announcement with the User DOC ready.

Thanks!
cc' @ren-odoo

[jcs-odoo]

Hi @vbe-odoo and @ren-odoo :)
I see Donatienne's review was a comment (not a change request) and gave a few pieces of advice here and there. So considering that this PR got already a few reviews, that Dona isn't available, and that it's urgent... Let's move forward with this PR ^^
(I haven't checked it)

[AntoineVDV]

@robodoo delegate+

[jcs-odoo]

@robodoo r+