Skip to content

Fix: platform specific identities#2837

Merged
ItzNotABug merged 1 commit intomainfrom
fix-identities
Feb 5, 2026
Merged

Fix: platform specific identities#2837
ItzNotABug merged 1 commit intomainfrom
fix-identities

Conversation

@ItzNotABug
Copy link
Member

@ItzNotABug ItzNotABug commented Feb 5, 2026

What does this PR do?

(Provide a description of what this PR does.)

Test Plan

(Write your test plan here. If you changed any code, please provide us with clear instructions on how you verified your changes work.)

Related PRs and Issues

(If this PR is related to any other PR or resolves any issue or related to any issue link all related PR and issues here.)

Have you read the Contributing Guidelines on issues?

(Write your answer here.)

Summary by CodeRabbit

Bug Fixes

  • Account identity management has been refined to automatically filter and exclude a specific provider type from the identities list displayed to users. This improvement ensures you see only applicable and relevant identity options available in your account settings, providing a clearer and more focused experience when managing your account identities.

@appwrite
Copy link

appwrite bot commented Feb 5, 2026

Console (appwrite/console)

Project ID: 688b7bf400350cbd60e9

Sites (1)
Site Status Logs Preview QR
 console-stage
688b7cf6003b1842c9dc
Ready Ready View Logs Preview URL QR Code

Tip

Appwrite has crossed the 50K GitHub stars milestone with hundreds of active contributors

@coderabbitai
Copy link
Contributor

coderabbitai bot commented Feb 5, 2026

Walkthrough

A filter constraint is added to the identities retrieval in the account layout route to exclude identities with the Imagine provider. The change imports Platform and Query and applies Query.notContains('provider', Platform.Imagine) to the existing identity list call. The overall load flow remains unchanged.

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~3 minutes

🚥 Pre-merge checks | ✅ 3
✅ Passed checks (3 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check ✅ Passed The title 'Fix: platform specific identities' is directly related to the changeset, which filters identities by excluding the Imagine provider platform.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.

✏️ Tip: You can configure your own custom pre-merge checks in the settings.

✨ Finishing touches
  • 📝 Generate docstrings
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Post copyable unit tests in a comment
  • Commit unit tests in branch fix-identities

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands and usage tips.

@ItzNotABug ItzNotABug merged commit 9e70b7c into main Feb 5, 2026
4 checks passed
@ItzNotABug ItzNotABug deleted the fix-identities branch February 5, 2026 08:51
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