-
Notifications
You must be signed in to change notification settings - Fork 10.7k
[ADD] accounting/l10n: new localization page for United Arab Emirates #2510
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
e91c139 to
5a3b90e
Compare
2b0657b to
1d22ad4
Compare
1d22ad4 to
dd70c79
Compare
f353340 to
55872e6
Compare
733de51 to
39e3a79
Compare
auva-odoo
left a comment
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 @toaa-odoo, I'm done with the review! I apologize in advance if some of my comments are particularly stupid; this was a difficult subject for me as a newbie. Anyways, we can talk about it whenerver you are ready :)
content/applications/finance/fiscal_localizations/united_arab_emirates.rst
Outdated
Show resolved
Hide resolved
content/applications/finance/fiscal_localizations/united_arab_emirates.rst
Show resolved
Hide resolved
content/applications/finance/fiscal_localizations/united_arab_emirates.rst
Outdated
Show resolved
Hide resolved
content/applications/finance/fiscal_localizations/united_arab_emirates.rst
Outdated
Show resolved
Hide resolved
content/applications/finance/fiscal_localizations/united_arab_emirates.rst
Outdated
Show resolved
Hide resolved
content/applications/finance/fiscal_localizations/united_arab_emirates.rst
Outdated
Show resolved
Hide resolved
content/applications/finance/fiscal_localizations/united_arab_emirates.rst
Outdated
Show resolved
Hide resolved
content/applications/finance/fiscal_localizations/united_arab_emirates.rst
Show resolved
Hide resolved
content/applications/finance/fiscal_localizations/united_arab_emirates.rst
Outdated
Show resolved
Hide resolved
content/applications/finance/fiscal_localizations/united_arab_emirates.rst
Outdated
Show resolved
Hide resolved
a4c37be to
41c196c
Compare
auva-odoo
left a comment
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.
@toaa-odoo just a few more things :)
content/applications/finance/fiscal_localizations/united_arab_emirates.rst
Outdated
Show resolved
Hide resolved
content/applications/finance/fiscal_localizations/united_arab_emirates.rst
Outdated
Show resolved
Hide resolved
content/applications/finance/fiscal_localizations/united_arab_emirates.rst
Outdated
Show resolved
Hide resolved
content/applications/finance/fiscal_localizations/united_arab_emirates.rst
Outdated
Show resolved
Hide resolved
content/applications/finance/fiscal_localizations/united_arab_emirates.rst
Outdated
Show resolved
Hide resolved
content/applications/finance/fiscal_localizations/united_arab_emirates.rst
Outdated
Show resolved
Hide resolved
content/applications/finance/fiscal_localizations/united_arab_emirates.rst
Outdated
Show resolved
Hide resolved
content/applications/finance/fiscal_localizations/united_arab_emirates.rst
Show resolved
Hide resolved
41c196c to
978392a
Compare
xpl-odoo
left a comment
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 @toaa-odoo ! Mostly nitpicks as usual. Good job overall :)
I'm pinging you @auva-odoo so you can see what you may have missed - even though some comments are based on personal preferences and not strict doc guidelines or grammar rules ;)
content/applications/finance/fiscal_localizations/united_arab_emirates.rst
Outdated
Show resolved
Hide resolved
content/applications/finance/fiscal_localizations/united_arab_emirates.rst
Outdated
Show resolved
Hide resolved
content/applications/finance/fiscal_localizations/united_arab_emirates.rst
Outdated
Show resolved
Hide resolved
content/applications/finance/fiscal_localizations/united_arab_emirates.rst
Outdated
Show resolved
Hide resolved
content/applications/finance/fiscal_localizations/united_arab_emirates.rst
Outdated
Show resolved
Hide resolved
content/applications/finance/fiscal_localizations/united_arab_emirates.rst
Outdated
Show resolved
Hide resolved
content/applications/finance/fiscal_localizations/united_arab_emirates.rst
Outdated
Show resolved
Hide resolved
content/applications/finance/fiscal_localizations/united_arab_emirates.rst
Outdated
Show resolved
Hide resolved
content/applications/finance/fiscal_localizations/united_arab_emirates.rst
Outdated
Show resolved
Hide resolved
content/applications/finance/fiscal_localizations/united_arab_emirates.rst
Outdated
Show resolved
Hide resolved
978392a to
d2908ef
Compare
taskid: 2906051
d2908ef to
0c65bfb
Compare
|
@robodoo r+ |
closes #2510 Taskid: 2906051 Signed-off-by: Platteau Xavier (xpl) <xpl@odoo.com>
task-2906051