Skip to content

Move & rename extras.webhooks_worker.process_webhook #14395

Closed
@jeremystretch

Description

@jeremystretch

Proposed Changes

Move the existing process_webhook() function, which is responsible for transmitting queued webhooks, from extras.webhooks_worker to extras.webhooks. I'll also propose renaming it to send_webhook(), which IMO better conveys its function.

We can retain a proxy for this function in its current location to provide backward compatibility until the next release. This will ensure any currently queued webhooks continue to be processed successfully after an upgrade.

Justification

Cleanup in the wake of #14132 and clarification of the function's purpose.

Metadata

Metadata

Assignees

Labels

status: acceptedThis issue has been accepted for implementationtype: housekeepingChanges to the application which do not directly impact the end user

Type

No type

Projects

No projects

Milestone

Relationships

None yet

Development

No branches or pull requests

Issue actions