-
Notifications
You must be signed in to change notification settings - Fork 219
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Update DelayJS to v2 #7092
Comments
I found this PR to merge v2 into develop then we can merge develop into main and release from main but I can see some conflicts, let me check if we can resolve the conflicts from our side or we will need to check with Gael. |
@wordpressfan This has already been discussed with Gael here: https://wp-media.slack.com/archives/CUT7FLHF1/p1729614432850799?thread_ts=1729184313.104719&cid=CUT7FLHF1 |
Awesome, so we will ignore conflicts and keep the v2 changes. |
Waiting feedback for possible regression https://wp-media.slack.com/archives/CUT7FLHF1/p1731051322877729?thread_ts=1729184313.104719&cid=CUT7FLHF1 |
Regression list
non-regression
|
CC @gmetais |
Hi @Mai-Saad, Could you please describe this issue a bit more?
What is a saved tab? Is it related to the fact that DelayJS intentionally loads scripts when the page visibility is hidden? |
@gmetais Here is a video about the saved tab (usually scripts aren't delayed there) |
DelayJS v2 must be released on WP Rocket.
The "v2" is currently available on the "v2" branch of the delay JS repo.
Here is the process to automatically update WP Rocket with a DelayJS release: https://www.notion.so/wpmedia/Maintaining-JavaScript-files-98e4112922d540da93e2536866e36fd8?pvs=4#95bb73a9447745228d1c6376e5ad3732
The text was updated successfully, but these errors were encountered: