Skip to content

Revert "Feat: Support appwrite oauth"#203

Merged
abnegate merged 1 commit into
mainfrom
revert-202-feat-appwrite-oauth
Jul 2, 2026
Merged

Revert "Feat: Support appwrite oauth"#203
abnegate merged 1 commit into
mainfrom
revert-202-feat-appwrite-oauth

Conversation

@abnegate

@abnegate abnegate commented Jul 2, 2026

Copy link
Copy Markdown
Member

Reverts #202

Copilot AI review requested due to automatic review settings July 2, 2026 10:25
@abnegate abnegate merged commit f6d5a1b into main Jul 2, 2026
5 checks passed
@abnegate abnegate deleted the revert-202-feat-appwrite-oauth branch July 2, 2026 10:25
@greptile-apps

greptile-apps Bot commented Jul 2, 2026

Copy link
Copy Markdown
Contributor

Greptile Summary

This PR reverts #202 by removing the appwrite provider entry from the PROVIDERS allow-list in OAuth2Provider.php.

  • Removes the single 'appwrite' => ['clientId' => ['target' => self::TARGET_APP_ID]] line; no other logic is affected.
  • Since fromArray silently returns null for unknown providers, any code path that previously attempted to migrate an Appwrite OAuth2 provider will now skip it gracefully.

Confidence Score: 5/5

Single-line revert with no logic changes; safe to merge.

The change removes one array entry from a static allow-list. The rest of the file is untouched, and callers gracefully handle unknown providers by returning null.

No files require special attention.

Important Files Changed

Filename Overview
src/Migration/Resources/Auth/OAuth2/OAuth2Provider.php Removes the 'appwrite' entry from the PROVIDERS allow-list, reverting PR #202's addition of Appwrite OAuth2 support.

Reviews (1): Last reviewed commit: "Revert "Feat: Support appwrite oauth"" | Re-trigger Greptile

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Pull request overview

This PR reverts the previously-added support for migrating the appwrite OAuth2 provider configuration by removing it from the OAuth2 provider allow-list. This aligns with the stated goal of reverting #202.

Changes:

  • Removed the appwrite entry from OAuth2Provider::PROVIDERS, meaning appwrite OAuth2 provider settings will no longer be considered safe/readable for migration.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

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