-
Notifications
You must be signed in to change notification settings - Fork 10.6k
[ADD] Payroll: Turkey localization #15237
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
5e7cd7b to
09bd9c6
Compare
|
Hello @Moah-odoo - this is ready for a look! Please let me know if there is anything that needs to be changed, or if I misunderstood any of the information you gave me. Thank you! |
Moah-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.
@larm-odoo A few comments and we will be good to go :)
content/applications/hr/payroll/payroll_localizations/turkey.rst
Outdated
Show resolved
Hide resolved
content/applications/hr/payroll/payroll_localizations/turkey.rst
Outdated
Show resolved
Hide resolved
content/applications/hr/payroll/payroll_localizations/turkey.rst
Outdated
Show resolved
Hide resolved
09bd9c6 to
4031262
Compare
4031262 to
d6d40be
Compare
|
Hello @Moah-odoo - thank you for the review! I would like you to double-check the gross-from-net and net-to-gross section you commented on. I only partially understood your comment, so I wasn't 100% sure how to edit the text. If I didn't get it right, would you let me know what it should say? Thank you! Once I get your approval, I will pass on to final review and we can publish! =) |
Moah-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.
@larm-odoo I tried explaining Net-to-Gross vs. Gross-to-Net. Let me know if it is clear now, or if you need me to add more context.
One part that I forgot to add in my previous explanation is:
The way taxes are computed is based on the amount of gross pay the employee has received up to the payslip date.
Example: In January, my total income year to date is 10K. I compute the income tax based on that 10K — let’s say that my tax amount is X. In February, my year-to-date gross is 20K, and my tax amount based on that is Y. But since I already paid X last month, this month I only need to pay Y − X, and so on.
Doing this results in the lowest amount of tax being deducted in January, and the highest amount being deducted in December, which means:
In Net-to-Gross: fixed net, gross increases throughout the year.
In Gross-to-Net: fixed gross, net decreases throughout the year.
d6d40be to
016cd77
Compare
|
Thank you @Moah-odoo - you made that easy to understand! I have implemented the changes to the two sections, so it is ready for you to take another look and approve. |
Moah-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.
@larm-odoo only 2 small changes and we will be good to go
content/applications/hr/payroll/payroll_localizations/turkey.rst
Outdated
Show resolved
Hide resolved
content/applications/hr/payroll/payroll_localizations/turkey.rst
Outdated
Show resolved
Hide resolved
016cd77 to
860ba37
Compare
|
Thank you @Moah-odoo - I changed the currency, so we should be good to go! If you agree, please approve, and I will move this to final/tech review, and we will get this published. =) |
Moah-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.
Good to go, many thanks. With this one, we have completed all current documentation for the MENA region. :D
|
Hi @Felicious - this is ready for a final/tech review. And as Mohamed said, this will be the last localization we need to make for the MENA region!!! |
Felicious
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.
LGTM @larm-odoo !
@robodoo delegate+
content/applications/hr/payroll/payroll_localizations/turkey.rst
Outdated
Show resolved
Hide resolved
860ba37 to
baf6e4b
Compare
|
@robodoo r+ |
closes #15237 Signed-off-by: Lara Martini (larm) <larm@odoo.com>

Doc does not exist. Requested to create the doc on this task card.
Original task card for this PR.