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.

Sign up for GitHub

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

Jump to bottom

Fix unload event used in core Javascript #2812

Open

brianhogg wants to merge 3 commits into dev from fix/unload-event-in-tracking

Contributor

brianhogg commented Nov 21, 2024

Description

Changes the unload event to pagehide

Fixes #2620

How has this been tested?

Manually

Checklist:

This PR requires and contains at least one changelog file.
My code has been tested.
My code passes all existing automated tests.
My code follows the LifterLMS Coding & Documentation Standards.

brianhogg added 2 commits

November 21, 2024 09:51


          Switching to use pagehide vs deprecated unload event.

d6c630a


          Removing build files.

4405f60

brianhogg requested a review from ideadude as a code owner

November 21, 2024 15:37


          Changelog.

32667fd

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet