Skip to content
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

Custom timings showing on pages that arent-related #500

Conversation

darcythomas
Copy link
Contributor

In some rare circumstance a custom timing may be created before the raygun4js has fully initialized.

In this case then event is queued for sending later.
In some very rare cases by the time the event is sent the page (and therefore the parent resource) has changed.

This can result in some timings being miss-attributed to the wrong page.

This pr tries to get the parent resource before the event is queued.

@darcythomas darcythomas force-pushed the darcy/WORM-253/custom-timings-showing-on-pages-that-arent-related branch from c4b94a3 to b799e9b Compare August 28, 2023 02:28
@darcythomas darcythomas changed the title Update raygun.js Custom timings showing on pages that arent-related Aug 28, 2023
Copy link
Collaborator

@Olwiba Olwiba left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Nice work on this one, make sure to validate we've covered all the calls to _publicRaygunFunctions.trackEvent to inject this new param if possible.

Otherwise code changes looking good 👍
I'm guessing the pre-soak has gone OK?

@darcythomas darcythomas merged commit 65a6098 into master Aug 30, 2023
1 check passed
darcythomas added a commit that referenced this pull request Aug 30, 2023
@xenolightning xenolightning mentioned this pull request Sep 12, 2023
darcythomas added a commit that referenced this pull request Sep 12, 2023
* Revert "Custom timings showing on pages that arent-related (#500)"

This reverts commit 65a6098.

* bump bump

* Update CHANGELOG.md
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants