[PR #14197] [MERGED] Closes #13645: Make Sentry integration optional #14331

Closed
opened 2025-12-29 23:23:52 +01:00 by adam · 0 comments
Owner

📋 Pull Request Information

Original PR: https://github.com/netbox-community/netbox/pull/14197
Author: @jeremystretch
Created: 11/6/2023
Status: Merged
Merged: 11/9/2023
Merged by: @jeremystretch

Base: featureHead: 13645-sentry-sdk-optional


📝 Commits (1)

📊 Changes

7 files changed (+29 additions, -34 deletions)

View changed files

📝 base_requirements.txt (+0 -4)
📝 docs/administration/error-reporting.md (+3 -15)
📝 docs/configuration/error-reporting.md (+3 -0)
📝 docs/installation/3-netbox.md (+11 -0)
📝 netbox/netbox/settings.py (+9 -12)
📝 netbox/netbox/views/errors.py (+3 -2)
📝 requirements.txt (+0 -1)

📄 Description

Fixes: #13645

  • Remove sentry-sdk from required packages list
  • Remove default DSN (maintained by community)
  • Update import statements
  • Add note to installation docs

🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.

## 📋 Pull Request Information **Original PR:** https://github.com/netbox-community/netbox/pull/14197 **Author:** [@jeremystretch](https://github.com/jeremystretch) **Created:** 11/6/2023 **Status:** ✅ Merged **Merged:** 11/9/2023 **Merged by:** [@jeremystretch](https://github.com/jeremystretch) **Base:** `feature` ← **Head:** `13645-sentry-sdk-optional` --- ### 📝 Commits (1) - [`21a9d16`](https://github.com/netbox-community/netbox/commit/21a9d16f7e9d6d5c9777017e15fee510e2b10cb0) Closes #13645: Make Sentry integration optional ### 📊 Changes **7 files changed** (+29 additions, -34 deletions) <details> <summary>View changed files</summary> 📝 `base_requirements.txt` (+0 -4) 📝 `docs/administration/error-reporting.md` (+3 -15) 📝 `docs/configuration/error-reporting.md` (+3 -0) 📝 `docs/installation/3-netbox.md` (+11 -0) 📝 `netbox/netbox/settings.py` (+9 -12) 📝 `netbox/netbox/views/errors.py` (+3 -2) 📝 `requirements.txt` (+0 -1) </details> ### 📄 Description ### Fixes: #13645 - Remove `sentry-sdk` from required packages list - Remove default DSN (maintained by community) - Update import statements - Add note to installation docs --- <sub>🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.</sub>
adam added the pull-request label 2025-12-29 23:23:52 +01:00
adam closed this issue 2025-12-29 23:23:53 +01:00
Sign in to join this conversation.
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: starred/netbox#14331