Skip to content

[18.0][FIX] report_xml: Remove useless and faulty post init hook#1153

Merged
OCA-git-bot merged 1 commit intoOCA:18.0from
acsone:18.0-report-xml-useless-hook
Apr 15, 2026
Merged

[18.0][FIX] report_xml: Remove useless and faulty post init hook#1153
OCA-git-bot merged 1 commit intoOCA:18.0from
acsone:18.0-report-xml-useless-hook

Conversation

@lmignon
Copy link
Copy Markdown
Contributor

@lmignon lmignon commented Apr 14, 2026

The post init hook created an ir attachment using content read as a string as value for the 'datas' field which expects a b64 encoded binary value.... This code comes from previous odoo version and should have been removed when migrating this addon

FWP #1152

The post init hook created an ir attachment using content read as a string as value for the 'datas' field which expects a b64 encoded binary value....
This code comes from previous odoo version and should have been removed when migrating this addon
@lmignon
Copy link
Copy Markdown
Contributor Author

lmignon commented Apr 15, 2026

/ocabot merge patch

@lmignon
Copy link
Copy Markdown
Contributor Author

lmignon commented Apr 15, 2026

fast tracking since initial pr is approved

@OCA-git-bot
Copy link
Copy Markdown
Contributor

This PR looks fantastic, let's merge it!
Prepared branch 18.0-ocabot-merge-pr-1153-by-lmignon-bump-patch, awaiting test results.

@OCA-git-bot OCA-git-bot merged commit d215096 into OCA:18.0 Apr 15, 2026
6 of 7 checks passed
@OCA-git-bot
Copy link
Copy Markdown
Contributor

Congratulations, your PR was merged at 8ec62da. Thanks a lot for contributing to OCA. ❤️

@lmignon lmignon deleted the 18.0-report-xml-useless-hook branch April 15, 2026 08:04
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.

3 participants