Closed fw-bot closed 1 month ago
@LucasLefevre @pro-odoo this PR targets master and is the last of the forward-port chain.
To merge the full chain, use
@robodoo r+
More info at https://github.com/odoo/odoo/wiki/Mergebot#forward-port
robodoo r+
On Wed, Aug 21, 2024 at 11:39 AM robodoo @.***> wrote:
[image: Pull request status dashboard] https://mergebot.odoo.com/odoo/o-spreadsheet/pull/4874
— Reply to this email directly, view it on GitHub https://github.com/odoo/o-spreadsheet/pull/4874#issuecomment-2301612209, or unsubscribe https://github.com/notifications/unsubscribe-auth/AHFIBHV6R7NVBH2QTDMSABDZSRN5BAVCNFSM6AAAAABM3UDBFSVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDGMBRGYYTEMRQHE . You are receiving this because you were mentioned.Message ID: @.***>
Steps to reproduce (in odoo):
This is a partial revert of 22625e8730ccc
It was actually intended to keep the pivot runtime definition when deleting the pivot core definition. The goal was to avoid redoing RPC to reload the pivot after an UNDO.
2 alternatives considered:
"REMOVE_PIVOT"
toUNDO_REDO_PIVOT_COMMANDS
to properly recreate the pivot. But spreadsheet pivot would be recreated twice :/ (and it'd be more RPCsDescription:
description of this task, what is implemented and why it is implemented that way.
Task: : TASK_ID
review checklist
Forward-Port-Of: odoo/o-spreadsheet#4863