mirror of
https://github.com/maziggy/bambuddy.git
synced 2026-05-09 08:25:54 +02:00
[GH-ISSUE #871] [Feature]: Standardize Webhook Notification Payload Fields #592
Labels
No labels
A1
automated
automated
bug
bug
Closed due to inactivity
contrib
dependencies
dependencies
duplicate
enhancement
feedback
hold
invalid
Notes
P1S
pull-request
security
ThumbsUp
user-report
wontfix
No milestone
No project
No assignees
1 participant
Notifications
Due date
No due date set.
Dependencies
No dependencies set.
Reference
starred/bambuddy-maziggy-1#592
Loading…
Add table
Add a link
Reference in a new issue
No description provided.
Delete branch "%!s()"
Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?
Originally created by @VREmma on GitHub (Apr 2, 2026).
Original GitHub issue: https://github.com/maziggy/bambuddy/issues/871
Originally assigned to: @maziggy on GitHub.
Problem or Use Case
While working on getting some notification webhook automation setup the last few days, I noticed that the documentation says that the webhook payload format has certain fields, but they are not always there for all notifications. I was looking to do some automation on the event field data, using information from the printer name and filename, etc as seen in the Payload format: https://wiki.bambuddy.cool/features/notifications/#webhook-custom
However, Print Started, Print Complete, First Layer Complete, and progress notifications all do not have these fields. They only have title, message, and the image. It also appears the Maintenance Due notifications only have title and message to them, no event or printer.
Proposed Solution
I propose we get the webhook notification payload standardized across all the notifications (where applicable). This way every payload has the basics of the print job/task on hand, and the webhooks can parse the data without having to parse the message field for any data. We can keep the title and message in there, and it should not break any current notifications since its just a json payload we are adding more to.
Alternatives Considered
No response
Feature Category
Notifications
Priority
Nice to have
Mockups or Examples
Here is an example from n8n that shows that Print Completed doesn't send the other payload fields like the documentation suggests.
Contribution
Checklist
@maziggy commented on GitHub (Apr 2, 2026):
Available/Fixed in branch dev and available with the next release or daily build.
Can you please test and let me know how it goes? Thanks!
@VREmma commented on GitHub (Apr 2, 2026):
Looks good so far! Just started 3 prints across 3 printers, and they all sent images and the other fields across the board so far. Will watch for it to progress through everything and make sure they are okay.