Skip to content

[IMP] Accounting - Remove l10n_us Payroll section - #19886

Closed
larm-odoo wants to merge 1 commit into
19.0from
19.0-accounting-remove-us-payroll-larn
Closed

[IMP] Accounting - Remove l10n_us Payroll section#19886
larm-odoo wants to merge 1 commit into
19.0from
19.0-accounting-remove-us-payroll-larn

Conversation

@larm-odoo

Copy link
Copy Markdown
Contributor

In  another task,  we are standardizing how Accounting localization pages link to their corresponding Payroll localizations. While doing that, we noticed that the US Accounting localization page still contains its Payroll information. Moving the payroll info out of the accounting section. and adding the more detailed ADP export section to the payroll documentation. 

Original task card for this PR.

@robodoo

robodoo commented Sep 1, 2026

Copy link
Copy Markdown
Collaborator

Pull request status dashboard

@larm-odoo larm-odoo self-assigned this Sep 1, 2026
@larm-odoo larm-odoo added the 2 label Sep 1, 2026
@larm-odoo
larm-odoo requested review from dade-odoo and jero-odoo and removed request for jero-odoo September 1, 2026 19:30
@larm-odoo

Copy link
Copy Markdown
Contributor Author

Hi @dade-odoo - would you like to review this, or should I leave it for our internal marketing team to review? Let me know! I think I removed everything needed- I left the direct deposit, but moved the DP export since that is only payroll.

@C3POdoo
C3POdoo requested review from a team September 1, 2026 19:33

@dade-odoo dade-odoo left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Hey @larm-odoo , I'll let your team review the payroll doc but I took a quick look at the accounting localization page. I'd just add the guilabel that I commented specifically and also add a See Also under the optional modules section that links to the payroll page.
Thanks!

Comment thread content/applications/finance/fiscal_localizations/united_states.rst Outdated
@larm-odoo
larm-odoo force-pushed the 19.0-accounting-remove-us-payroll-larn branch 2 times, most recently from dbdb1e0 to 3e9bdc9 Compare September 2, 2026 14:27
@larm-odoo
larm-odoo requested a review from erjer-odoo September 2, 2026 14:35
@larm-odoo

Copy link
Copy Markdown
Contributor Author

Hi @erjer-odoo - this is ready for a quick final approval. I moved a section out of the accounting doc to the payroll doc. However, the US payroll localization doc does not need to be reviewed, as I have other edits to make (a sort of overhaul), and I'd prefer to manage all of that at once in a single PR. This is only for the accounting page, and really for only the changes requested. I did not do an overhaul for formatting on the accounting page since that is out of our scope, and I didn't want to step on any toes. Thank you!

@erjer-odoo erjer-odoo left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Just two small changes!

@robodoo delegate+

Comment thread content/applications/hr/payroll/payroll_localizations/united_states.rst Outdated
Comment thread content/applications/hr/payroll/payroll_localizations/united_states.rst Outdated
@larm-odoo
larm-odoo force-pushed the 19.0-accounting-remove-us-payroll-larn branch from 3e9bdc9 to 9cb09ec Compare September 2, 2026 20:38
@larm-odoo

Copy link
Copy Markdown
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.

4 participants