Skip to content

[MOV] Payroll: moving employment hero #13849

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

Closed
wants to merge 1 commit into from

Conversation

larm-odoo
Copy link
Contributor

@larm-odoo larm-odoo commented Jun 26, 2025

Moving the Payroll Hero doc from the Finance section to HR> Payroll>  Payroll Localization

Task card for this PR, which was requested by @dade-odoo.

NOTE- The images and information must be updated in a separate PR, this is for the structure/order only. I did change any use of the word "you".

@robodoo
Copy link
Collaborator

robodoo commented Jun 26, 2025

Pull request status dashboard

@larm-odoo larm-odoo requested review from samueljlieber and a team and removed request for samueljlieber June 26, 2025 14:20
@C3POdoo C3POdoo requested review from a team June 26, 2025 14:41
@larm-odoo
Copy link
Contributor Author

Hi @samueljlieber - per Dallas's request in Discord, this PR is ot move the Payroll Hero doc out of the Finance section and into the Payroll Localization folder in the HR section. Since this is not knowledge-based, and this is a structural move, I am sending this straight to you for technical review. Dallas, or myself if I learn about this more, will update the doc for technical information in a future PR. This is only to address the move.

image

@larm-odoo larm-odoo self-assigned this Jun 26, 2025
@larm-odoo larm-odoo added the 1 label Jun 26, 2025
@Felicious
Copy link
Contributor

Hi @larm-odoo ! Since this is a move, can you add redirect links too? And this PR is worth 2 pts

@larm-odoo
Copy link
Contributor Author

Hi @larm-odoo ! Since this is a move, can you add redirect links too? And this PR is worth 2 pts

I FORGOT ABOUT THE REDIRECTS =D =D =D

@larm-odoo larm-odoo force-pushed the 18.0-payroll-move-employment-hero branch from bfc6865 to f863e7c Compare June 27, 2025 21:30
@larm-odoo larm-odoo added 2 and removed 1 labels Jun 27, 2025
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.

found a typo in the redirect, but after fixing that, it's ready @larm-odoo !

@robodoo delegate=larm-odoo

@larm-odoo larm-odoo force-pushed the 18.0-payroll-move-employment-hero branch from f863e7c to 4b49cb8 Compare June 27, 2025 22:48
@larm-odoo
Copy link
Contributor Author

@robodoo r+

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.

3 participants