Skip to content

[FIX] tests: allow exporting the POT files again#1816

Closed
fw-bot wants to merge 1 commit intoodoo:masterfrom
odoo-dev:master-saas-18.4-fix-industry-pot-export-dyki-504720-fw
Closed

[FIX] tests: allow exporting the POT files again#1816
fw-bot wants to merge 1 commit intoodoo:masterfrom
odoo-dev:master-saas-18.4-fix-industry-pot-export-dyki-504720-fw

Conversation

@fw-bot
Copy link

@fw-bot fw-bot commented Mar 4, 2026

The trans_export function was changed to require an environment instead of a cursor as its last argument. This made it so the test failed to export the POT files.

We update the function call here to use the correct argument.

Forward-Port-Of: #1801

The `trans_export` function was changed to require an environment
instead of a cursor as its last argument. This made it so the test
failed to export the POT files.

We update the function call here to use the correct argument.

X-original-commit: 7510c15
@robodoo
Copy link
Collaborator

robodoo commented Mar 4, 2026

Pull request status dashboard

@fw-bot
Copy link
Author

fw-bot commented Mar 4, 2026

@fw-bot
Copy link
Author

fw-bot commented Mar 4, 2026

@dylankiss @vava-odoo ci/industry failed on this forward-port PR

@vava-odoo
Copy link
Collaborator

@robodoo r+

@robodoo
Copy link
Collaborator

robodoo commented Mar 4, 2026

@vava-odoo you may want to rebuild or fix this PR as it has failed CI.

@vava-odoo
Copy link
Collaborator

@robodoo override=ci/industry

robodoo pushed a commit that referenced this pull request Mar 4, 2026
The `trans_export` function was changed to require an environment
instead of a cursor as its last argument. This made it so the test
failed to export the POT files.

We update the function call here to use the correct argument.

closes #1816

X-original-commit: 7510c15
Signed-off-by: Vallaeys Valentin (vava) <vava@odoo.com>
@robodoo robodoo added the 19.3 label Mar 4, 2026
@robodoo robodoo closed this Mar 4, 2026
@fw-bot fw-bot deleted the master-saas-18.4-fix-industry-pot-export-dyki-504720-fw branch March 11, 2026 20:36
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants