Skip to content

[18.0] [IMP] survey_xlsx: add extension hooks to the XLSX report - #261

Open
mondot wants to merge 1 commit into
OCA:18.0from
elabore-coop:18.0-imp-survey_xlsx-hooks
Open

mondot wants to merge 1 commit into
OCA:18.0from
elabore-coop:18.0-imp-survey_xlsx-hooks

Conversation

@mondot

@mondot mondot commented Sep 23, 2026

Copy link
Copy Markdown

Extract the per-question header writing, the per-answer processing and a per-user-input post-processing step into dedicated methods called by generate_xlsx_report. This is behavior-preserving and lets downstream modules customize the export without overriding the whole method.

Assisted-By: Claude Opus 4.8 noreply@anthropic.com

@mondot

mondot commented Sep 23, 2026 •

Copy link
Copy Markdown
Author

Hi @rousseldenis, here is the PR I talked about earlier with you at OCA Days.
Basically it's only extracting methods to be able to customize the export.

Please let me know if it's enough to be merged :)

@mondot
mondot marked this pull request as draft September 23, 2026 13:51
Extract the per-question header writing, the per-answer processing and a
per-user-input post-processing step into dedicated methods called by
generate_xlsx_report. This is behavior-preserving and lets downstream
modules customize the export without overriding the whole method.

Assisted-By: Claude Opus 4.8 <noreply@anthropic.com>
@mondot
mondot force-pushed the 18.0-imp-survey_xlsx-hooks branch from 8c80346 to e00f88e Compare September 23, 2026 13:56
@mondot
mondot marked this pull request as ready for review September 23, 2026 13:57
@mondot mondot changed the title [IMP] survey_xlsx: add extension hooks to the XLSX report [18.0] [IMP] survey_xlsx: add extension hooks to the XLSX report Sep 23, 2026
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.

2 participants