Add a description field to the Webhook model #8887

Closed
opened 2025-12-29 20:42:31 +01:00 by adam · 3 comments
Owner

Originally created by @jeremystretch on GitHub (Nov 28, 2023).

Originally assigned to: @jeremystretch on GitHub.

NetBox version

v3.6.5

Feature type

Data model extension

Proposed functionality

Add a standard description field to the Webhook model.

Use case

This will enable users to record a brief description concerning the use of the webhook. This has become more important in the wake of #14267, which separates the event logic from the webhook definition using the new EventRule model.

Database changes

Add a new CharField

External dependencies

None

Originally created by @jeremystretch on GitHub (Nov 28, 2023). Originally assigned to: @jeremystretch on GitHub. ### NetBox version v3.6.5 ### Feature type Data model extension ### Proposed functionality Add a standard description field to the Webhook model. ### Use case This will enable users to record a brief description concerning the use of the webhook. This has become more important in the wake of #14267, which separates the event logic from the webhook definition using the new EventRule model. ### Database changes Add a new CharField ### External dependencies None
adam added the status: acceptedtype: feature labels 2025-12-29 20:42:31 +01:00
adam closed this issue 2025-12-29 20:42:31 +01:00
Author
Owner

@jacobpfickes commented on GitHub (Nov 29, 2023):

I'd like to contribute to this issue

@jacobpfickes commented on GitHub (Nov 29, 2023): I'd like to contribute to this issue
Author
Owner

@jeremystretch commented on GitHub (Nov 29, 2023):

@jacobpfickes I really appreciate the interest and normally I'd be happy to assign this to you, however we're in a bit of a time crunch right now as we need to release the v3.7 beta by end of week. There might also be some conflicts with other ongoing work that could complicate the PR process. So I'm going to knock this out myself real quick, but I encourage you to browse some of our other open issues to see if there's anything else you'd like to take on.

@jeremystretch commented on GitHub (Nov 29, 2023): @jacobpfickes I really appreciate the interest and normally I'd be happy to assign this to you, however we're in a bit of a time crunch right now as we need to release the v3.7 beta by end of week. There might also be some conflicts with other ongoing work that could complicate the PR process. So I'm going to knock this out myself real quick, but I encourage you to browse some of our other [open issues](https://github.com/netbox-community/netbox/issues?q=is%3Aissue+is%3Aopen+label%3A%22status%3A+needs+owner%22) to see if there's anything else you'd like to take on.
Author
Owner

@jacobpfickes commented on GitHub (Nov 29, 2023):

@jeremystretch Ok completely understand. I have a changes on my fork and ready to submit a PR right now if that helps. If not then no worries. It is my fault for not getting it assigned before working on it.

@jacobpfickes commented on GitHub (Nov 29, 2023): @jeremystretch Ok completely understand. I have a changes on my fork and ready to submit a PR right now if that helps. If not then no worries. It is my fault for not getting it assigned before working on it.
Sign in to join this conversation.
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: starred/netbox#8887