odoo / o-spreadsheet

Other
193 stars 44 forks source link

[FW][FIX] find & replace: remove debounce from store #4880

Closed fw-bot closed 1 month ago

fw-bot commented 1 month ago

Description

The method updateSearchContent was debounced inside the f&r store. This does not work as expected, as the re-rendering of the panel was done when calling the updateSearchContent method, and not when the debounced function was called and the store state was updated.

I'm not sure why it still worked 50% of the time.

Task: : 4102172

review checklist

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

robodoo commented 1 month ago

Pull request status dashboard

fw-bot commented 1 month ago

@hokolomopo @LucasLefevre this PR targets master and is the last of the forward-port chain containing:

To merge the full chain, use

@robodoo r+

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

LucasLefevre commented 1 month ago

robodoo r+

On Thu, Aug 22, 2024 at 9:25 AM Otis - Odoo Forward-Port Steward [bot] < @.***> wrote:

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

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/4880#issuecomment-2303966982, or unsubscribe https://github.com/notifications/unsubscribe-auth/ACTHFEJ64OYLX5VOWQWI22LZSWG7BAVCNFSM6AAAAABM5QQDTSVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDGMBTHE3DMOJYGI . You are receiving this because you were mentioned.Message ID: @.***>