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.

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

Update + Reconfigure Vue Sentry SDK #1795

Merged
merged 5 commits into from
Dec 20, 2023
Merged

Update + Reconfigure Vue Sentry SDK #1795

merged 5 commits into from
Dec 20, 2023

Conversation

mvandenburgh
Copy link
Member

I noticed this error log appearing in each Sentry issue for the frontend -

[@sentry/vue]: Misconfigured SDK. Vue specific errors will not be captured.
Update your `Sentry.init` call with an appropriate config option:
`app` (Application Instance - Vue 3) or `Vue` (Vue Constructor - Vue 2).

So, I updated the sentry SDK to the latest version and reconfigured it using the latest docs. I also enabled Sentry Replay so we can try that out.

@mvandenburgh
Copy link
Member Author

Hmm, the deploy preview is still logging that SDK error message in the console 🤔

@mvandenburgh mvandenburgh marked this pull request as draft December 19, 2023 15:25
@mvandenburgh
Copy link
Member Author

It looks like this was the issue 🤦

@mvandenburgh mvandenburgh marked this pull request as ready for review December 19, 2023 20:21
@mvandenburgh mvandenburgh added patch Increment the patch version when merged release Create a release when this pr is merged labels Dec 20, 2023
@mvandenburgh mvandenburgh merged commit 0cc5847 into master Dec 20, 2023
3 checks passed
@mvandenburgh mvandenburgh deleted the vue-sentry-update branch December 20, 2023 19:51
@dandibot
Copy link
Member

🚀 PR was released in v0.3.68 🚀

@dandibot dandibot added the released This issue/pull request has been released. label Dec 20, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@danlamanna danlamanna danlamanna approved these changes

@jjnesbitt jjnesbitt Awaiting requested review from jjnesbitt

Assignees
No one assigned
Labels
patch Increment the patch version when merged release Create a release when this pr is merged released This issue/pull request has been released.
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@mvandenburgh @dandibot @danlamanna