Skip to content

feat(api): add workspace_slug to webhook delivery payload#9232

Open
ch4og wants to merge 1 commit into
makeplane:previewfrom
ch4og:preview
Open

feat(api): add workspace_slug to webhook delivery payload#9232
ch4og wants to merge 1 commit into
makeplane:previewfrom
ch4og:preview

Conversation

@ch4og

@ch4og ch4og commented Jun 12, 2026

Copy link
Copy Markdown
Contributor

Description

Add workspace_slug to the webhook delivery payload. The slug wasn't included in the payload. Webhook users can now build workspace URLs or make API calls. Before it was impossible even with API call since there is no way to get slug from workspace_id.

Type of Change

  • Bug fix (non-breaking change which fixes an issue)
  • Feature (non-breaking change which adds functionality)
  • Improvement (change that would cause existing functionality to not work as expected)
  • Code refactoring
  • Performance improvements
  • Documentation update

Screenshots and Media (if applicable)

Test Scenarios

Verified webhook payload now includes workspace_slug field alongside workspace_id.
Confirmed HMAC signature still computes correctly.

References

@coderabbitai

coderabbitai Bot commented Jun 12, 2026

Copy link
Copy Markdown
Contributor

Review Change Stack

📝 Walkthrough

Walkthrough

Single-line change adding workspace_slug field to the webhook JSON payload in the webhook_send_task function, complementing existing workspace_id and webhook_id fields.

Changes

Webhook Payload Enhancement

Layer / File(s) Summary
Webhook payload with workspace slug
apps/api/plane/bgtasks/webhook_task.py
The webhook request JSON body gains a workspace_slug field derived from the slug argument, included in the payload sent to configured webhook endpoints.

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~2 minutes

Poem

🐰 A slug hops into the webhook's way,
Workspace whispers what it has to say,
JSON grows one field more,
Payloads knock on webhook doors! 📬

🚥 Pre-merge checks | ✅ 5
✅ Passed checks (5 passed)
Check name Status Explanation
Title check ✅ Passed The title accurately summarizes the main change: adding workspace_slug to the webhook payload, which is directly supported by the code change.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.
Linked Issues check ✅ Passed Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check ✅ Passed Check skipped because no linked issues were found for this pull request.
Description check ✅ Passed The PR description covers all required sections from the template with appropriate details and test scenarios.

✏️ Tip: You can configure your own custom pre-merge checks in the settings.

✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands and usage tips.

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