-
Notifications
You must be signed in to change notification settings - Fork 619
Sentry config fixes for dashboard #5056
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
Sentry config fixes for dashboard #5056
Conversation
|
|
The latest updates on your projects. Learn more about Vercel for Git ↗︎
|
Your org has enabled the Graphite merge queue for merging into mainAdd the label “merge-queue” to the PR and Graphite will automatically add it to the merge queue when it’s ready to merge. Or use the label “hotfix” to add to the merge queue as a hot fix. You must have a Graphite account and log in to Graphite in order to use the merge queue. Sign up using this link. |
|
This stack of pull requests is managed by Graphite. Learn more about stacking. Join @MananTank and the rest of your teammates on |
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## main #5056 +/- ##
=======================================
Coverage 45.29% 45.29%
=======================================
Files 1059 1059
Lines 54724 54724
Branches 3953 3953
=======================================
Hits 24786 24786
Misses 29247 29247
Partials 691 691
*This pull request uses carry forward flags. Click here to find out more. |
size-limit report 📦
|
ddf7d0b to
39053f7
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
.
56adae8 to
2ad062c
Compare
2ad062c to
86304f1
Compare
86304f1 to
be79c9b
Compare
d00ec05 to
2f01f9a
Compare
2f01f9a to
8e5ca0c
Compare
8e5ca0c to
2bc913f
Compare
Merge activity
|
## Problem solved
Short description of the bug fixed or feature added
<!-- start pr-codex -->
---
## PR-Codex overview
This PR updates the configuration for `Sentry` in the `next.config.js` file and modifies the `turbo.json` to include an environment variable for `SENTRY_AUTH_TOKEN`. It also refines the `Sentry` setup options for better functionality and logging.
### Detailed summary
- Added `"env": ["SENTRY_AUTH_TOKEN"]` to `turbo.json`.
- Updated `Sentry` configuration in `apps/dashboard/next.config.js`:
- Changed `silent` from `true` to `false`.
- Added `authToken: process.env.SENTRY_AUTH_TOKEN`.
- Retained other `Sentry` options with no changes.
> ✨ Ask PR-Codex anything about this PR by commenting with `/codex {your question}`
<!-- end pr-codex -->
2bc913f to
85e7d15
Compare

Problem solved
Short description of the bug fixed or feature added
PR-Codex overview
This PR focuses on enhancing the
Sentryconfiguration in thenext.config.jsfile and updating theturbo.jsonfile to include an environment variable forSENTRY_AUTH_TOKEN.Detailed summary
env:["SENTRY_AUTH_TOKEN"]inturbo.json.wSentryconfiguration inapps/dashboard/next.config.js:authToken: process.env.SENTRY_AUTH_TOKEN.silentfromtruetofalse.