Skip to content

Improved custom webhook documentation - #243

Draft
Dboy0ZDev wants to merge 2 commits into
mainfrom
webhook-integration
Draft

Improved custom webhook documentation#243
Dboy0ZDev wants to merge 2 commits into
mainfrom
webhook-integration

Conversation

@Dboy0ZDev

Copy link
Copy Markdown
Collaborator

Improved the custom webhook documentation with details, commands and examples for requests and responses. Special focus on requirements for tokens and requests.

@l3monKenji
l3monKenji self-requested a review August 12, 2026 11:49

@l3monKenji l3monKenji Aug 12, 2026

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This can be prettier

Image Image Image Image

Copy link
Copy Markdown
Collaborator Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Yes, saw this too and asked @Ph4t3 about it. Seems to be a dwt issue that is already fixed in the next version of dwt. Project would need to be updated, but @seb-kw usually does that.

@Dboy0ZDev

Copy link
Copy Markdown
Collaborator Author

@l3monKenji

I have added adjusted some template to clarify where the user is suppose to enter his own values.

All template commands have been manually tested.

@l3monKenji

Copy link
Copy Markdown
Collaborator

Are the command line suggestions tested and integrated into the testing suite?
If not please test them and give me a heads up, when they're tested, so i can review and test them as well.

If there is no test, please open up an issue to integrate the creation of webhooks into our testing suite.
Thank you :)

@Dboy0ZDev
Dboy0ZDev marked this pull request as draft August 14, 2026 16:10
@Dboy0ZDev

Copy link
Copy Markdown
Collaborator Author

Command line suggestions have been tested manually and they work. Currently working on updating the test-suite + adjusting the documentation page with screenshots of the web interface workflow.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants