Skip to content

Revise webhook documentation with additional details #585

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
Fawas003 merged 1 commit into from
Mar 18, 2026
+12 −2
Merged

Revise webhook documentation with additional details #585

Changes from all commits
Commits
Show all changes
1 commit
Select commit
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
14 changes: 12 additions & 2 deletions docs/4. Product Features/03. Flows/2. Flow Actions/11. Call a webhook.md
View file
Open in desktop
Original file line number Diff line number Diff line change
@@ -1,5 +1,13 @@
> ### **3 minutes read                                                                                                                         `Advanced`**
___

<h3>
<table>
<tr>
<td><b>3 minutes read</b></td>
<td style={{ paddingLeft: '40px' }}><b>Level: Advanced </b></td>
<td style={{ paddingLeft: '40px' }}><b>Last Updated: March 2026</b></td>
</tr>
</table>
</h3>
Comment on lines +2 to +10
Copy link
Contributor

@coderabbitai coderabbitai bot Mar 18, 2026

Choose a reason for hiding this comment

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

⚠️ Potential issue | 🟡 Minor

Semantic HTML violation: table wrapped in heading tag.

Wrapping a <table> inside an <h3> element is semantically incorrect. Heading elements should contain text content, not structural elements like tables. This can negatively impact accessibility (screen readers may misinterpret the document structure) and SEO.

Consider removing the <h3> wrapper or using a <div> with appropriate styling instead.

♻️ Proposed fix to improve semantic HTML 
-<h3>
- <table>
+<div>
+<table>
   <tr>
     <td><b>3 minutes read</b></td>
     <td style={{ paddingLeft: '40px' }}><b>Level: Advanced </b></td>
     <td style={{ paddingLeft: '40px' }}><b>Last Updated: March 2026</b></td>
   </tr>
 </table>
-</h3>
+</div>

If the <h3> styling is needed, apply it via a CSS class on the <div> instead.

📝 Committable suggestion

‼️ IMPORTANT
Carefully review the code before committing. Ensure that it accurately replaces the highlighted code, contains no missing lines, and has no issues with indentation. Thoroughly test & benchmark the code to ensure it meets the requirements.

Suggested change
<h3>
<table>
<tr>
<td><b>3 minutes read</b></td>
<td style={{ paddingLeft: '40px' }}><b>Level: Advanced </b></td>
<td style={{ paddingLeft: '40px' }}><b>Last Updated: March 2026</b></td>
</tr>
</table>
</h3>
<div>
<table>
<tr>
<td><b>3 minutes read</b></td>
<td style={{ paddingLeft: '40px' }}><b>Level: Advanced </b></td>
<td style={{ paddingLeft: '40px' }}><b>Last Updated: March 2026</b></td>
</tr>
</table>
</div>
🤖 Prompt for AI Agents 
Verify each finding against the current code and only fix it if needed.

In `@docs/4`. Product Features/03. Flows/2. Flow Actions/11. Call a webhook.md
around lines 2 - 10, The heading currently wraps a table (<h3> containing
<table>), which is semantically incorrect; remove the <h3> wrapper and either
place the <table> directly in the document with a preceding real heading element
(e.g., an <h3> containing text) or replace the <h3> with a <div> that carries
the heading styling (apply a CSS class) so the structure uses a proper heading
element separate from the table and the layout/styling is preserved.


**Webhook calls are used to communicate with third-party applications within flows. They allow NGOs to receive and handle incoming messages, delivery receipts and other events from WhatsApp in a seamless and automated manner.**

Expand Down Expand Up @@ -103,4 +111,6 @@ ___

- [Glific Webhooks](https://glific.github.io/slate/#webhooks)

- [List of HTTP status codes](https://en.wikipedia.org/wiki/List_of_HTTP_status_codes)

- [Code examples to use Webhooks in Glific](https://github.com/glific/recipes)
Loading