- Notifications
You must be signed in to change notification settings - Fork11.2k
feat: filter out platform organizations from admin organization list#24926
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to ourterms of service andprivacy statement. We’ll occasionally send you account related emails.
Already on GitHub?Sign in to your account
feat: filter out platform organizations from admin organization list#24926
Uh oh!
There was an error while loading.Please reload this page.
Conversation
Co-Authored-By: sean@cal.com <Sean@brydon.io>
🤖 Devin AI EngineerI'll be helping with this pull request! Here's what you should know: ✅ I will automatically:
Note: I can only respond to comments from users who have write access to this repository. ⚙️ Control Options:
|
This PR has been marked as stale due to inactivity. If you're still working on it or need any help, please let us know or update the PR to keep it active. |
vercelbot commentedNov 21, 2025 • edited
Loading Uh oh!
There was an error while loading.Please reload this page.
edited
Uh oh!
There was an error while loading.Please reload this page.
The latest updates on your projects. Learn more aboutVercel for GitHub. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others.Learn more.
No issues found across 1 file
3c8215a intomainUh oh!
There was an error while loading.Please reload this page.
E2E results are ready! |
What does this PR do?
Filters out platform organizations from the admin organization list by adding
isPlatform: falseto the query filter inadminGetAll.handler.ts.Requested by:sean@cal.com (@sean-brydon)
Devin Session:https://app.devin.ai/sessions/9c26a1afb0a2496a80f5427c9d0de950
Changes Made
isPlatform: falsefilter to theadminGetAllhandler to exclude platform organizations from the admin organization list view_optsparameterHow should this be tested?
/settings/admin/organizationsisPlatform: trueare no longer visible in the listisPlatform: falseor null) are still displayed correctlyNote: This change was not tested locally. Manual verification is recommended.
Important Review Notes
isPlatformis false). Please confirm this is the correct interpretation.Mandatory Tasks (DO NOT REMOVE)
Checklist