Closed fw-bot closed 2 months ago
@LucasLefevre @rrahir cherrypicking of pull request odoo/o-spreadsheet#4727 failed.
stdout:
Auto-merging src/state_observer.ts
CONFLICT (content): Merge conflict in src/state_observer.ts
Either perform the forward-port manually (and push to this branch, proceeding as usual) or close this PR (maybe?).
In the former case, you may want to edit this PR message as well.
:warning: after resolving this conflict, you will need to merge it via @robodoo.
More info at https://github.com/odoo/odoo/wiki/Mergebot#forward-port
Description:
The history is sometimes used in plugin
import
methods:this.history.update(...)
There are 2 main reason to do that:
this.history.update("myData", sheetId, pivotId, value)
, ifmyData
is empty, it will creates intermediary objects with keyssheetId
andpivotId
:myData: { [sheetId]: { [pivotId]: data } }
At import however, keeping track of the changes is useless. They are currently aggregated in the
changes
array, but never used.This commit improves loading time of the large number data set by ~17%
Before: 1353ms After: 1116ms
(averages over 8 runs) Task: : 4083517
review checklist
Forward-Port-Of: odoo/o-spreadsheet#4727