fix: remove unsupported Query.select causing invalid queries param (#2916)#2919
fix: remove unsupported Query.select causing invalid queries param (#2916)#2919bugbotbyte wants to merge 2 commits intoappwrite:mainfrom
Conversation
Console (appwrite/console)Project ID: Sites (1)
Tip HTTPS and SSL certificates are handled automatically for all your Sites |
|
No actionable comments were generated in the recent review. 🎉 ℹ️ Recent review info⚙️ Run configurationConfiguration used: Organization UI Review profile: CHILL Plan: Pro Run ID: 📒 Files selected for processing (1)
🚧 Files skipped from review as they are similar to previous changes (1)
WalkthroughThe pull request updates two console-route files. In +layout.ts the project list query was simplified to call Estimated code review effort🎯 2 (Simple) | ⏱️ ~10 minutes 🚥 Pre-merge checks | ✅ 5✅ Passed checks (5 passed)
✏️ Tip: You can configure your own custom pre-merge checks in the settings. ✨ Finishing Touches🧪 Generate unit tests (beta)
📝 Coding Plan
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. Comment Tip You can make CodeRabbit's review stricter and more nitpicky using the `assertive` profile, if that's what you prefer.Change the |
There was a problem hiding this comment.
Actionable comments posted: 1
🤖 Prompt for all review comments with AI agents
Verify each finding against the current code and only fix it if needed.
Inline comments:
In `@src/routes/`(console)/organization-[organization]/createProject.svelte:
- Line 39: The invalidate call after creating a project (await
invalidate(Dependencies.ORGANIZATION)) can throw and currently lives inside the
same try that reports mutation success/failure; move it out or wrap it in its
own try/catch so invalidate errors don’t trigger the main catch and incorrectly
mark project creation as failed—specifically adjust the code around the
mutation/try block in createProject.svelte to handle the mutation success
separately from calling invalidate(Dependencies.ORGANIZATION), and log or ignore
errors from invalidate without rethrowing.
ℹ️ Review info
⚙️ Run configuration
Configuration used: Organization UI
Review profile: CHILL
Plan: Pro
Run ID: f7c73f00-fddd-4a06-8e35-de4859f1d8df
📒 Files selected for processing (2)
src/routes/(console)/+layout.tssrc/routes/(console)/organization-[organization]/createProject.svelte

What does this PR do?
Fixes the "Invalid queries param: Invalid query method: select" error when listing projects.
This occurs because
Query.select()is not supported for the/projectsendpoint in Appwrite 1.8.Changes
Query.select(['$id'])from the projects query in+layout.tsinvalidate(Dependencies.ORGANIZATION)after project creation to refresh the project list so the new project appears immediatelyRelated Issue
Closes #2916
Summary by CodeRabbit
Bug Fixes
Improvements