[FW][FIX] pivot: update pivot formulas when data set changes

fw-bot commented 2 months ago

Description:

Steps to reproduce:

create a pivot based on a range
remove the =PIVOT(1) formula
add a few individual PIVOT.VALUE or PIVOT.HEADER formula
change values in the range data set => the formulas are not updated

Task: : 4082765

review checklist

[ ] feature is organized in plugin, or UI components
[ ] support of duplicate sheet (deep copy)
[ ] in model/core: ranges are Range object, and can be adapted (adaptRanges)
[ ] in model/UI: ranges are strings (to show the user)
[ ] undo-able commands (uses this.history.update)
[ ] multiuser-able commands (has inverse commands and transformations where needed)
[ ] new/updated/removed commands are documented
[ ] exportable in excel
[ ] translations (_t("qmsdf %s", abc))
[ ] unit tested
[ ] clean commented code
[ ] track breaking changes
[ ] doc is rebuild (npm run doc)
[ ] status is correct in Odoo

Forward-Port-Of: odoo/o-spreadsheet#4725

robodoo commented 2 months ago

fw-bot commented 2 months ago

@LucasLefevre @rrahir 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

LucasLefevre commented 2 months ago

robodoo r+

On Fri, Aug 2, 2024 at 2:42 PM Otis - Odoo Forward-Port Steward [bot] < @.***> wrote:

@LucasLefevre https://github.com/LucasLefevre @rrahir https://github.com/rrahir this PR targets master and is the last of the forward-port chain.

To merge the full chain, use

@robodoo https://github.com/robodoo r+

More info at https://github.com/odoo/odoo/wiki/Mergebot#forward-port

— Reply to this email directly, view it on GitHub https://github.com/odoo/o-spreadsheet/pull/4771#issuecomment-2265294099, or unsubscribe https://github.com/notifications/unsubscribe-auth/ACTHFEL6XMB2VQGHMEP6MR3ZPN5CZAVCNFSM6AAAAABL4PEIG2VHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDENRVGI4TIMBZHE . You are receiving this because you were mentioned.Message ID: @.***>

robodoo commented 2 months ago

@LucasLefevre @rrahir 'ci/runbot' failed on this reviewed PR.

fw-bot commented 2 months ago

@LucasLefevre @rrahir this PR was modified / updated and has become a normal PR. It must be merged directly.

LucasLefevre commented 2 months ago

robodoo r+

odoo / o-spreadsheet

[FW][FIX] pivot: update pivot formulas when data set changes #4771

Description:

review checklist