Skip to content

Conversation

@larm-odoo
Copy link
Contributor

Batches have been replaced by "pay runs" so all nomenclature, buttons, etc require updating. 

Original task card for this PR

@larm-odoo larm-odoo self-assigned this Dec 11, 2025
@robodoo
Copy link
Collaborator

robodoo commented Dec 11, 2025

Pull request status dashboard

@larm-odoo larm-odoo requested a review from jero-odoo December 11, 2025 20:03
@larm-odoo larm-odoo added the 3 label Dec 11, 2025
@C3POdoo C3POdoo requested a review from a team December 11, 2025 20:05
@larm-odoo
Copy link
Contributor Author

Hi @jero-odoo - this is ready for a peer review. Thanks!

@larm-odoo larm-odoo force-pushed the 19.0-payroll-new-pay-runs-larm branch from 4304584 to f59c325 Compare December 12, 2025 17:57
Copy link
Contributor

@jero-odoo jero-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 @larm-odoo only a few edits, also a few notes:
-Make sure you do the redirect (since it looks like you are removing the other doc)

  • There are two images that have not been compressed (work-entries-pay-run and employees)
    Approving for the next round!

@larm-odoo larm-odoo force-pushed the 19.0-payroll-new-pay-runs-larm branch from f59c325 to 3868232 Compare December 12, 2025 21:58
@larm-odoo larm-odoo requested a review from Felicious December 12, 2025 21:58
@larm-odoo
Copy link
Contributor Author

Hi @Felicious - this is ready for a final review!

Copy link
Contributor

@Felicious Felicious left a comment

Choose a reason for hiding this comment

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

LGTM @larm-odoo !

Just caught an extra space between a link and a comma 🤣 Good work as always!

@robodoo delegate+

Process a pay run
=================

After a :ref:`pay run has been created <payroll/new-pay-run>` , and :ref:`all required payslips have
Copy link
Contributor

Choose a reason for hiding this comment

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

Suggested change
After a :ref:`pay run has been created <payroll/new-pay-run>` , and :ref:`all required payslips have
After a :ref:`pay run has been created <payroll/new-pay-run>`, and :ref:`all required payslips have

@Felicious Felicious added 5 and removed 3 labels Dec 12, 2025
@Felicious
Copy link
Contributor

Also I increased the scope of the PR from 3 to 5, because a new file was created. SINCE a new file was created though, please create a redirect link!

https://github.com/odoo/documentation/blob/19.0/redirects/MANUAL.md

Copy link
Contributor

@Felicious Felicious left a comment

Choose a reason for hiding this comment

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

Forgot to ask for redirect!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants