zammad-admin-documentation
zammad-admin-documentation copied to clipboard
Improved webhook logs documentation with debugging and testing details
- Expanded details on webhook request and response attributes.
- Added a Debugging Webhook Issues section to help diagnose failures.
- Improved readability and structure for better comprehension.
Hi @ssganesh035,
thanks a lot for your contribution! However, your PR should be slightly tweaked:
- Please select
pre-releaseas merge target branch - Execute a
make gettextto update the translation catalog and commit them - Rephrase line 30+ to something like:
Before deploying webhooks in production, you can use testing tools to inspect and debug requests. If you do so, make sure to exclude any sensitive or personal data.And then add the example services.
Hi @ralf401, hope you are doing well. I have commited the suggested changes, I believe it's good to go now.
Not yet. It seems you just changed the base branch and didn't rebase/update it to the current pre-release branch.
Please do that and make sure to execute the make gettext before committing. The PR may have no conflict before running the CI checks.
Thank you!
Hi @ssganesh035, any news about that? Feel free to rebase and push. Alternatively, I could create a new PR with your suggested changes.
Hi @ralf401, I will do the needful and update you
Hi @ssganesh035, what about the update of your PR? I understood that you wanted to updated it, right?
Hey @ralf401, hope you are doing well. Could you create a new PR with the suggested changes, as I am unable to execute make gettext. Thanks!
Hi @ralf401, kindly let me know if there's an update
@ssganesh035 not yet. I'll take a look in the next days...
Cool, thanks!
Hi @ssganesh035 I created a PR here and close your PR now. Thanks for your contribution!