Skip to content

Comments

docs(credentials): replace environment variables page with credentials docs#3331

Merged
waleedlatif1 merged 1 commit intostagingfrom
docs/creds
Feb 25, 2026
Merged

docs(credentials): replace environment variables page with credentials docs#3331
waleedlatif1 merged 1 commit intostagingfrom
docs/creds

Conversation

@waleedlatif1
Copy link
Collaborator

Summary

  • Removed the old environment variables docs page (all languages)
  • Added new credentials section with docs for secrets, OAuth accounts, workspace vs personal scoping, access control
  • Added screenshots for creating secrets, OAuth connections, using {{}} references, and the block credential selector

Type of Change

  • Documentation update

Testing

Tested manually

Checklist

  • Code follows project style guidelines
  • Self-reviewed my changes
  • Tests added/updated and passing
  • No new warnings introduced
  • I confirm that I have read and agree to the terms outlined in the Contributor License Agreement (CLA)

@vercel
Copy link

vercel bot commented Feb 25, 2026

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Actions Updated (UTC)
docs Building Building Preview, Comment Feb 25, 2026 7:57am

Request Review

@greptile-apps
Copy link
Contributor

greptile-apps bot commented Feb 25, 2026

Greptile Summary

This PR replaces the outdated environment variables documentation with comprehensive credentials documentation. The new docs expand coverage to include both secrets (static API keys) and OAuth accounts (authenticated service connections), with detailed explanations of workspace vs personal scoping, access control, bulk import functionality, and best practices.

Key improvements:

  • More accurate terminology: "credentials" better describes the feature than "environment variables"
  • OAuth accounts documentation added (previously not covered)
  • Access control and role-based permissions explained
  • Visual documentation with 6 new screenshots showing the UI flow
  • Clearer explanation of resolution order and scoping behavior
  • Added bulk import documentation for .env-style content

Changes made:

  • Removed old environment-variables.mdx files across all 6 language directories (de, en, es, fr, ja, zh)
  • Added new apps/docs/content/docs/en/credentials/index.mdx with comprehensive documentation
  • Updated meta.json to include credentials section in navigation
  • Added 6 supporting screenshots to illustrate credential management workflows

The old environment variable images in /static/environment/ are now unused but not deleted in this PR.

Confidence Score: 5/5

  • This PR is safe to merge - it's a documentation-only change with no code modifications
  • Documentation update with proper content structure, valid image references, and clean removal of outdated files. No code changes, no broken links, and all screenshots properly added.
  • No files require special attention

Important Files Changed

Filename Overview
apps/docs/content/docs/en/credentials/index.mdx New credentials documentation with comprehensive coverage of secrets, OAuth, scoping, and access control
apps/docs/content/docs/en/meta.json Added credentials section to navigation
apps/docs/content/docs/en/variables/environment-variables.mdx Removed old environment variables documentation (replaced by credentials docs)

Last reviewed commit: b7ab397

Copy link
Contributor

@greptile-apps greptile-apps bot left a comment

Choose a reason for hiding this comment

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

14 files reviewed, no comments

Edit Code Review Agent Settings | Greptile

@waleedlatif1 waleedlatif1 merged commit ff01825 into staging Feb 25, 2026
12 checks passed
@waleedlatif1 waleedlatif1 deleted the docs/creds branch February 25, 2026 08:02
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