Skip to content

feat variables api#2915

Open
atharvadeosthale wants to merge 3 commits intomainfrom
worktree-variables-api
Open

feat variables api#2915
atharvadeosthale wants to merge 3 commits intomainfrom
worktree-variables-api

Conversation

@atharvadeosthale
Copy link
Copy Markdown
Member

No description provided.

@greptile-apps
Copy link
Copy Markdown
Contributor

greptile-apps Bot commented Apr 27, 2026

Greptile Summary

This PR adds the Variables API documentation: a new blog post, changelog entry, project-level environment variables doc page, expanded function and site env-vars pages, a new functions-env-vars.md partial, and the associated screenshots.

  • The blog post and the changelog entry both reference cover: /images/blog/announcing-variables-api/cover.png, but this image file was not included in the PR and does not exist under static/. Every other announcement post ships its cover image in the same PR — without it both pages will render a broken image on publish.

Confidence Score: 4/5

Safe to merge after adding the missing cover image asset; all documentation content and SDK examples are otherwise consistent.

One P1 finding: the cover image referenced by both the blog post and the changelog is absent from the repository, causing a broken image on two public pages at publish time. No logic or security issues were found in the documentation content itself.

src/routes/blog/post/announcing-variables-api/+page.markdoc and src/routes/changelog/(entries)/2026-04-28.markdoc — both reference a missing static asset.

Important Files Changed

Filename Overview
src/routes/blog/post/announcing-variables-api/+page.markdoc New blog post for the Variables API announcement; references a cover image that is absent from the repository.
src/routes/changelog/(entries)/2026-04-28.markdoc New changelog entry for 2026-04-28; references the missing cover image; scope names are consistent with the rest of the PR.
src/routes/docs/advanced/platform/environment-variables/+page.markdoc New project-level env-vars doc page with full SDK multi-language examples; uses newly added platform screenshots and correct scope names.
src/routes/docs/products/functions/environment-variables/+page.markdoc Expanded functions env-vars doc with SDK CRUD examples; previous reviewer flagged a missing import os in the Python Read variables snippet.
src/routes/docs/products/sites/environment-variables/+page.markdoc New sites env-vars doc page; images reference existing static assets and SDK examples look consistent.

Reviews (3): Last reviewed commit: "fix ids" | Re-trigger Greptile

Comment thread src/routes/changelog/(entries)/2026-04-28.markdoc
```server-nodejs
import { Client, Project, ID } from 'node-appwrite';

const client = new Client()
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

P1 Missing blog cover image

The cover field references /images/blog/announcing-variables-api/cover.png, but no such file was added in this PR and it does not exist under static/images/blog/. Every other announcement post (e.g. announcing-webhooks-api, announcing-bulk-api) ships the cover image as part of the same PR. Without it, the blog post and the changelog entry will both render a broken image.

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