Skip to content

Conversation

@micheledic
Copy link

@micheledic micheledic commented Nov 24, 2025

Fwport of #1095

@micheledic micheledic changed the title [FW][FIX] report_xslx: Revert action_manager_report.esm.js to fix KeyError on OCA Financial Reports. [FW][19.0][FIX] report_xlsx : error exporting xlsx fwport of #1095 Nov 24, 2025
@pedrobaeza pedrobaeza added this to the 19.0 milestone Nov 24, 2025
@pedrobaeza
Copy link
Member

Please cherry-pick the existing commit as is, or revert the 19.0 original commit, but don't recreate it for not mangling commit history for tools like oca-port.

@micheledic
Copy link
Author

micheledic commented Nov 24, 2025

Please cherry-pick the existing commit as is, or revert the 19.0 original commit, but don't recreate it for not mangling commit history for tools like oca-port.

I only cherry-picked of the specific commit of the other PR, i have not changed anything.. on files changed there is only the javascript modified by other pr

@pedrobaeza
Copy link
Member

OK, sorry. I saw it the bad way.

/ocabot merge patch

@OCA-git-bot
Copy link
Contributor

This PR looks fantastic, let's merge it!
Prepared branch 19.0-ocabot-merge-pr-1099-by-pedrobaeza-bump-patch, awaiting test results.

@OCA-git-bot
Copy link
Contributor

@pedrobaeza The merge process could not be finalized, because command git push origin 19.0-ocabot-merge-pr-1099-by-pedrobaeza-bump-patch:19.0 failed with output:

To https://github.com/OCA/reporting-engine
 ! [remote rejected]   19.0-ocabot-merge-pr-1099-by-pedrobaeza-bump-patch -> 19.0 (cannot lock ref 'refs/heads/19.0': is at 51a998f15094c79551de54f7a962f315513ee2f9 but expected 116c643a646bc79a05fdea83ee60ac822a00717f)
error: failed to push some refs to 'https://github.com/OCA/reporting-engine'

@OCA-git-bot OCA-git-bot merged commit 27e74ad into OCA:19.0 Nov 24, 2025
7 checks passed
@OCA-git-bot
Copy link
Contributor

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

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