-
Notifications
You must be signed in to change notification settings - Fork 10.4k
[IMP] l10n_id: Improve Indonesia localization module documentation #14773
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
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Hi @XIDO-odoo! Thank you for your work on this Indonesian loc page!
I added some comments/suggestions. Let me know if you have any questions.
Thanks
content/applications/finance/fiscal_localizations/indonesia.rst
Outdated
Show resolved
Hide resolved
content/applications/finance/fiscal_localizations/indonesia.rst
Outdated
Show resolved
Hide resolved
content/applications/finance/fiscal_localizations/indonesia.rst
Outdated
Show resolved
Hide resolved
content/applications/finance/fiscal_localizations/indonesia.rst
Outdated
Show resolved
Hide resolved
content/applications/finance/fiscal_localizations/indonesia.rst
Outdated
Show resolved
Hide resolved
content/applications/finance/fiscal_localizations/indonesia.rst
Outdated
Show resolved
Hide resolved
content/applications/finance/fiscal_localizations/indonesia.rst
Outdated
Show resolved
Hide resolved
3418006 to
fa12b34
Compare
|
Hello @afma-odoo , I made the revisions and 1 comment, could you please have a check? If any other revisions are necessary, please just let me know. Thank you! |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Hi @XIDO-odoo Thank you for the update! I just left one more suggestion in the Product section.
Could you also add a custom anchor for the Company section? Thanks!
I'll go ahead and approve the PR.
Once the last updates are finished, I'll r+ it ;)
Thank you!
content/applications/finance/fiscal_localizations/indonesia.rst
Outdated
Show resolved
Hide resolved
fa12b34 to
da04132
Compare
|
Hello @afma-odoo , thank you for the review. Could you please help a quick check about the anchor of the company and the whole doc? Thank you a lot! |
da04132 to
1cb5151
Compare
|
Hi @XIDO-odoo, thank you! I force-pushed some changes to add anchors to all sections 😉 @robodoo r+ |
closes #14773 Signed-off-by: Anne-Françoise Marcq (afma) <afma@odoo.com>
|
Hello @afma-odoo , thank you! |

Following the release of the improvement on Indonesia localization, this PR includes additional information to the page