Skip to content

feat: add project scopes to API keys#3009

Merged
atharvadeosthale merged 2 commits intomainfrom
feat-add-project-platforms-key-scopes
Apr 29, 2026
Merged

feat: add project scopes to API keys#3009
atharvadeosthale merged 2 commits intomainfrom
feat-add-project-platforms-key-scopes

Conversation

@atharvadeosthale
Copy link
Copy Markdown
Member

@atharvadeosthale atharvadeosthale commented Apr 29, 2026

Summary

  • Adds project.read and project.write to the API key scopes catalog (src/lib/constants.ts).
  • Both land under the Other category in the API key creation/edit UI.
  • Scopes already exist server-side in appwrite/app/config/scopes/project.php — this just exposes them in the console.

Test plan

  • Open Project → Overview → API keys → Create key
  • Expand the Other accordion
  • Verify project.read and project.write checkboxes appear with correct labels and descriptions
  • Create a key with the new scopes selected; confirm it persists and the API call succeeds

@greptile-apps
Copy link
Copy Markdown
Contributor

greptile-apps Bot commented Apr 29, 2026

Greptile Summary

Adds project.read and project.write ScopeDefinition entries to the API key scopes catalog, making them selectable in the Other accordion of the key creation/edit UI. The PR description also commits to adding platforms.read and platforms.write, but those two entries are missing from scopes: ScopeDefinition[] and therefore won't appear in the UI.

Confidence Score: 4/5

Safe to merge only after adding the missing platforms.read and platforms.write ScopeDefinition entries, or updating the PR description to reflect the reduced scope.

Single P1 finding: platforms.read and platforms.write are stated in the PR description but not implemented in the ScopeDefinition[] array, leaving users unable to select them in the UI.

src/lib/constants.ts — missing two ScopeDefinition objects for platforms scopes

Important Files Changed

Filename Overview
src/lib/constants.ts Adds project.read and project.write ScopeDefinition entries; platforms.read and platforms.write described in the PR are absent from the ScopeDefinition[] array.

Reviews (2): Last reviewed commit: "remove platforms scopes" | Re-trigger Greptile

Comment thread src/lib/constants.ts
@atharvadeosthale atharvadeosthale changed the title feat: add project and platforms scopes to API keys feat: add project scopes to API keys Apr 29, 2026
@atharvadeosthale atharvadeosthale merged commit dc5598c into main Apr 29, 2026
3 of 4 checks passed
@atharvadeosthale atharvadeosthale deleted the feat-add-project-platforms-key-scopes branch April 29, 2026 08:11
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