740dd9c6fa
Add server status as a default field in all webhooks using BaseWebhookData struct ( #4410 )
...
* Initial plan
* Add server status as default field in all webhooks
Co-authored-by: gabek <414923+gabek@users.noreply.github.com >
* Fix goimports linter error by removing trailing whitespace
Co-authored-by: gabek <414923+gabek@users.noreply.github.com >
* Move serverURL from status object to separate webhook field per feedback
Per code review feedback, serverURL is a configuration value, not a status property. This change:
- Removes ServerURL from models.Status struct
- Adds ServerURL as separate field in WebhookEvent
- Populates ServerURL directly when sending webhooks using configrepository.GetServerURL()
- Updates all tests to expect new structure
This provides the same functionality (server URL in all webhooks) while correctly treating it as configuration rather than status.
Co-authored-by: gabek <414923+gabek@users.noreply.github.com >
* Add omitempty tag to ServerURL field in WebhookEvent struct
Co-authored-by: gabek <414923+gabek@users.noreply.github.com >
* Fix webhook duplication by moving status to eventData for all events
Co-authored-by: gabek <414923+gabek@users.noreply.github.com >
* Restore type safety to webhook EventData using proper typed structs
Co-authored-by: gabek <414923+gabek@users.noreply.github.com >
* Move ServerURL from top-level WebhookEvent to eventData for all webhook types
Co-authored-by: gabek <414923+gabek@users.noreply.github.com >
* Update core/webhooks/webhooks.go
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com >
* Create BaseWebhookData struct for common webhook fields using struct embedding
Co-authored-by: gabek <414923+gabek@users.noreply.github.com >
---------
Co-authored-by: copilot-swe-agent[bot] <198982749+Copilot@users.noreply.github.com >
Co-authored-by: gabek <414923+gabek@users.noreply.github.com >
Co-authored-by: Gabe Kangas <gabek@real-ity.com >
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com >
2025-07-31 17:12:38 -07:00
Gabe Kangas and GitHub
0b5d7c8a4d
Config repository ( #3988 )
...
* WIP
* fix(test): fix ap test failing
* fix: fix unkeyed fields being used
* chore(tests): clean up browser tests by splitting out federation UI tests
2024-11-15 19:20:58 -08:00
Gabe Kangas and GitHub
209756fed3
feat(webhooks): add stream status to webhook ( #2934 )
...
Expand the payload sent for stream status webhooks. Closes #2881
2023-05-30 11:32:05 -07:00
Matthew Donoughe and GitHub
10055664bb
add tests for webhook events ( #2180 )
...
* add tests for webhook events
* atomic.Uint32 is not in Go 1.18
2022-10-09 22:55:54 -07:00
