Skip to content

Fix message when a resource triggers errors - #1341

Merged
Jethro Nederhof (jethron) merged 1 commit into
snowplow:release/4.0.0from
jethron:error-clarification
Sep 10, 2024
Merged

Fix message when a resource triggers errors#1341
Jethro Nederhof (jethron) merged 1 commit into
snowplow:release/4.0.0from
jethron:error-clarification

Conversation

@jethron

Copy link
Copy Markdown
Contributor

The error event fires on the window both when an unhandled JavaScript exception is thrown or a resource fails to load. Rather than track a message presuming the former, attempt to detect the latter and specify the failed resource.

See also:

See also:
BCPF-1456

@bundlemon

bundlemon Bot commented Sep 6, 2024

Copy link
Copy Markdown

BundleMon

Files added (6)
Status Path Size Limits
trackers/javascript-tracker/dist/sp.js
+27.81KB 30KB / +10%
libraries/browser-tracker-core/dist/index.mod
ule.js
+22.49KB 25KB / +10%
libraries/tracker-core/dist/index.module.js
+19.12KB 20KB / +10%
trackers/javascript-tracker/dist/sp.lite.js
+16.88KB 20KB / +10%
trackers/browser-tracker/dist/index.umd.min.j
s
+16.87KB 20KB / +10%
trackers/browser-tracker/dist/index.module.js
+3.49KB 5KB / +10%

Total files change +106.66KB 0%

Final result: ✅

View report in BundleMon website ➡️


Current branch size history

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

LGTM! 👍

Comment thread plugins/browser-plugin-error-tracking/src/index.ts Outdated
Comment thread plugins/browser-plugin-error-tracking/src/index.ts
The `error` event fires on the `window` both when an unhandled JavaScript
exception is thrown or a resource fails to load. Rather than track a message
presuming the former, attempt to detect the latter and specify the
failed resource.

See also:
- https://developer.mozilla.org/en-US/docs/Web/API/Window/error_event
- https://developer.mozilla.org/en-US/docs/Web/API/HTMLElement/error_event
@jethron
Jethro Nederhof (jethron) merged commit 335f05a into snowplow:release/4.0.0 Sep 10, 2024
@jethron
Jethro Nederhof (jethron) deleted the error-clarification branch September 10, 2024 03:46
Matus Tomlein (matus-tomlein) pushed a commit that referenced this pull request Oct 25, 2024
The `error` event fires on the `window` both when an unhandled JavaScript
exception is thrown or a resource fails to load. Rather than track a message
presuming the former, attempt to detect the latter and specify the
failed resource.

See also:
- https://developer.mozilla.org/en-US/docs/Web/API/Window/error_event
- https://developer.mozilla.org/en-US/docs/Web/API/HTMLElement/error_event
Matus Tomlein (matus-tomlein) pushed a commit that referenced this pull request Oct 28, 2024
The `error` event fires on the `window` both when an unhandled JavaScript
exception is thrown or a resource fails to load. Rather than track a message
presuming the former, attempt to detect the latter and specify the
failed resource.

See also:
- https://developer.mozilla.org/en-US/docs/Web/API/Window/error_event
- https://developer.mozilla.org/en-US/docs/Web/API/HTMLElement/error_event
Matus Tomlein (matus-tomlein) pushed a commit that referenced this pull request Oct 28, 2024
The `error` event fires on the `window` both when an unhandled JavaScript
exception is thrown or a resource fails to load. Rather than track a message
presuming the former, attempt to detect the latter and specify the
failed resource.

See also:
- https://developer.mozilla.org/en-US/docs/Web/API/Window/error_event
- https://developer.mozilla.org/en-US/docs/Web/API/HTMLElement/error_event
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Development

Successfully merging this pull request may close these issues.

2 participants