Skip to content

Fix GitHub OAuth label from App ID to Client ID#2978

Open
swetalin-10 wants to merge 2 commits intoappwrite:mainfrom
swetalin-10:fix-github-oauth-label
Open

Fix GitHub OAuth label from App ID to Client ID#2978
swetalin-10 wants to merge 2 commits intoappwrite:mainfrom
swetalin-10:fix-github-oauth-label

Conversation

@swetalin-10
Copy link
Copy Markdown

Fixes #11875

Updated the GitHub OAuth label from "App ID" to "Client ID" to match GitHub terminology and improve clarity.
Other providers remain unchanged.

@greptile-apps
Copy link
Copy Markdown
Contributor

greptile-apps bot commented Apr 14, 2026

Greptile Summary

This PR renames the "App ID" label to "Client ID" for GitHub and GitHub Enterprise OAuth providers in mainOAuth.svelte to match GitHub's official terminology. The change is minimal and correct, also picking up githubenterprise alongside github.

Confidence Score: 5/5

  • Safe to merge — the change is a label-only fix with no logic or runtime impact.
  • All findings are P2 style suggestions (restoring the type import modifier, and optionally renaming "App Secret" → "Client Secret" for GitHub). Neither blocks merge.
  • No files require special attention.

Important Files Changed

Filename Overview
src/routes/(console)/project-[region]-[project]/auth/(providers)/mainOAuth.svelte Label for the App ID field updated to "Client ID" for github and githubenterprise providers; type modifier dropped from the Provider import (P2 style). The parallel "App Secret" label was not updated to match GitHub's "Client Secret" terminology.

Reviews (2): Last reviewed commit: "fix: handle GitHub Enterprise OAuth labe..." | Re-trigger Greptile

Comment thread src/routes/(console)/project-[region]-[project]/auth/(providers)/mainOAuth.svelte Outdated
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