Skip to content

Add webhook scopes#2969

Merged
atharvadeosthale merged 1 commit intomainfrom
atharva/webhook-scopes
Apr 9, 2026
Merged

Add webhook scopes#2969
atharvadeosthale merged 1 commit intomainfrom
atharva/webhook-scopes

Conversation

@atharvadeosthale
Copy link
Copy Markdown
Member

Adds webhooks.read and webhooks.write scopes to console

@appwrite
Copy link
Copy Markdown

appwrite bot commented Apr 9, 2026

Console (appwrite/console)

Project ID: 688b7bf400350cbd60e9

Sites (1)
Site Status Logs Preview QR
 console-stage
688b7cf6003b1842c9dc
Building Building View Logs Preview URL QR Code

Tip

Preview deployments create instant URLs for every branch and commit

@greptile-apps
Copy link
Copy Markdown
Contributor

greptile-apps bot commented Apr 9, 2026

Greptile Summary

Adds webhooks.read and webhooks.write entries to the scopes array in src/lib/constants.ts, making these scopes selectable in the API keys UI. The entries follow the existing pattern for category: 'Other' with icon: 'globe', consistent with other miscellaneous scopes like migrations, tokens, and locale.

Confidence Score: 5/5

Safe to merge — minimal, additive change consistent with established patterns.

The change is two new constant entries that mirror the exact structure of a dozen similar entries already present. No logic, rendering, or type-safety concerns found; the canWriteWebhooks derived store in roles.ts already references webhooks.write, confirming the scope string is expected by the runtime.

No files require special attention.

Vulnerabilities

No security concerns identified.

Important Files Changed

Filename Overview
src/lib/constants.ts Adds webhooks.read and webhooks.write scope definitions to the scopes array, consistent with existing patterns for category 'Other' and icon 'globe'.

Reviews (1): Last reviewed commit: "Add webhook scopes" | Re-trigger Greptile

@atharvadeosthale atharvadeosthale merged commit 1b2598a into main Apr 9, 2026
3 of 4 checks passed
@atharvadeosthale atharvadeosthale deleted the atharva/webhook-scopes branch April 9, 2026 15:05
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