Skip to content
This repository has been archived by the owner on Sep 12, 2024. It is now read-only.

Track JS error - {${some error}} #455

Draft
wants to merge 3 commits into
base: master
Choose a base branch
from

Conversation

maryia-matskevich-deriv
Copy link
Contributor

@maryia-matskevich-deriv maryia-matskevich-deriv commented Apr 15, 2024

Changes:

Track JS error - {${some error}}
fix: get a full detailed error log of "unhandledrejection"

1) In the unhandledrejection event I added the code to extract the actual error.
2) Since the error happens in third-party load resources I've handled the most weak places in our code. I've added "wrappers" (block try/catch) for some methods of Google Drive API. I found articles where people use it for them.
3) The first thing that can hit the unhandledrejection event place is the 'throw e' line of code, so I replaced it with the trackJS() handler.

Screenshots:

Copy link

vercel bot commented Apr 15, 2024

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Updated (UTC)
binary-bot-on-deriv ✅ Ready (Inspect) Visit Preview Apr 23, 2024 6:12am

Copy link

Quality Gate Passed Quality Gate passed

Issues
0 New issues
0 Accepted issues

Measures
0 Security Hotspots
No data about Coverage
0.0% Duplication on New Code

See analysis details on SonarCloud

Copy link

Preview Link: https://pr-455.binary-bot-preview.pages.dev

Name Result
Build status Completed ✅
Preview URL Visit Preview
Action URL Visit Action

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant