Skip to content

Conversation

pipu-odoo
Copy link
Contributor

In this commit, we force the value of checkDelay to 50ms. checkDelay is the time after which we will look for the trigger in the DOM if there is no more mutation. By default, this value is 750ms (see macro.js). However, the tricks listed in this PR are able to run at 50ms which allows to reduce the execution time of these tricks.

Description of the issue/feature this PR addresses:

Current behavior before PR:

Desired behavior after PR is merged:


I confirm I have signed the CLA and read the PR guidelines at www.odoo.com/submit-pr

In this commit, we force the value of checkDelay to 50ms. checkDelay
is the time after which we will look for the trigger in the DOM if
there is no more mutation. By default, this value is 750ms
(see macro.js). However, the tricks listed in this PR are able to run
at 50ms which allows to reduce the execution time of these tricks.
@robodoo
Copy link
Contributor

robodoo commented Sep 23, 2024

Pull request status dashboard

@C3POdoo C3POdoo requested review from a team and caburj and removed request for a team September 23, 2024 08:35
@C3POdoo C3POdoo added the RD research & development, internal work label Sep 23, 2024
@BastienFafchamps
Copy link
Contributor

@robodoo r+

robodoo pushed a commit that referenced this pull request Sep 23, 2024
In this commit, we force the value of checkDelay to 50ms. checkDelay
is the time after which we will look for the trigger in the DOM if
there is no more mutation. By default, this value is 750ms
(see macro.js). However, the tricks listed in this PR are able to run
at 50ms which allows to reduce the execution time of these tricks.

closes #181003

Signed-off-by: Bastien Fafchamps (bafa) <bafa@odoo.com>
@robodoo robodoo closed this Sep 23, 2024
@robodoo robodoo added the 17.5 label Sep 23, 2024
@fw-bot fw-bot deleted the master-pos_checkdelay_50-pipu branch October 7, 2024 12:11
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

17.5 RD research & development, internal work

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants