Skip to content

feat(webhooks): query arguments for vote webhooks#10

Merged
velddev merged 2 commits into
mainfrom
feat/webhooks/query-parameter
May 28, 2026
Merged

feat(webhooks): query arguments for vote webhooks#10
velddev merged 2 commits into
mainfrom
feat/webhooks/query-parameter

Conversation

@velddev
Copy link
Copy Markdown
Contributor

@velddev velddev commented May 28, 2026

No description provided.

Copy link
Copy Markdown

@gemini-code-assist gemini-code-assist Bot left a comment

Choose a reason for hiding this comment

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

Code Review

This pull request updates the webhook events documentation in webhooks/events.mdx to include the new data.query object, which contains parsed query parameters appended to the vote page URL. A review comment was received pointing out a grammatical issue (a comma splice and capitalization error) in the description of data.query and providing a suggestion to improve readability.

Comment thread webhooks/events.mdx Outdated
| `data.project.type` | string | Project type (`"bot"` or `"server"`) |
| `data.project.platform` | string | Platform (always `"discord"`) |
| `data.project.platform_id` | string | Discord client ID of the bot or guild ID of the server |
| `data.query` | object | Parsed query parameters appended to the `/:id/vote` page URL, For example, `/:id/vote?key1=value&key2=value2` sends the parsed query arguments in `data.query`. |
Copy link
Copy Markdown

Choose a reason for hiding this comment

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

medium

There is a comma splice and a capitalization error in the description of data.query. It should use a period to separate the two independent clauses for better readability.

| `data.query` | object | Parsed query parameters appended to the `/:id/vote` page URL. For example, `/:id/vote?key1=value&key2=value2` sends the parsed query arguments in `data.query`. |

@velddev velddev merged commit b87d163 into main May 28, 2026
@velddev velddev deleted the feat/webhooks/query-parameter branch May 28, 2026 07:01
@mintlify
Copy link
Copy Markdown

mintlify Bot commented May 28, 2026

Preview deployment for your docs. Learn more about Mintlify Previews.

Project Status Preview Updated (UTC)
topgg 🔴 Failed May 28, 2026, 7:05 AM

💡 Tip: Enable Workflows to automatically generate PRs for you.

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.

1 participant