search

OCA / reporting-engine

Odoo Alternative Reporting Engine
https://odoo-community.org/project/tools-maintainers-30
GNU Affero General Public License v3.0
335 stars 795 forks source link

[16.0][ADD] report_text_format_option #884

Closed AungKoKoLin1997 closed 5 months ago

AungKoKoLin1997 commented 5 months ago

This module is designed to add encoding fields to the ir.actions.report model and is applied to text type reports.

This module was initially proposed in https://github.com/OCA/reporting-engine/pull/864, and report_csv depends on it. According to the discussion in this PR, this module should be used separately and specifically target text reports. Other modules (e.g., report_xml, report_csv) have their own encoding fields, and adding this module as a dependency could disrupt existing installations.

In this PR, I changed the module name from report_format_option to report_text_format_option to better reflect its primary focus on text report formatting. Additionally, I adjusted the encoding-related field names to avoid duplication with the report_csv encoding fields and to prevent breaking the UI.

@qrtl

OCA-git-bot commented 5 months ago

This PR has the approved label and has been created more than 5 days ago. It should therefore be ready to merge by a maintainer (or a PSC member if the concerned addon has no declared maintainer). 🤖

HviorForgeFlow commented 5 months ago

/ocabot merge nobump

OCA-git-bot commented 5 months ago

This PR looks fantastic, let's merge it! Prepared branch 16.0-ocabot-merge-pr-884-by-HviorForgeFlow-bump-nobump, awaiting test results.

OCA-git-bot commented 5 months ago

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