-
Notifications
You must be signed in to change notification settings - Fork 202
Update Console to handle specifications for self-hosted #2685
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 our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
base: main
Are you sure you want to change the base?
Conversation
Console (appwrite/console)Project ID: Tip Dynamic API keys are generated automatically for each function execution |
|
Important Review skippedBot user detected. To trigger a single review, invoke the You can disable this status message by setting the Comment |
|
@copilot try again |
- Modified functions layout loader to check isCloud before calling listSpecifications - Modified sites settings page loader to check isCloud before calling listSpecifications - Updated function create pages to handle empty specifications array safely - Updated updateResourceLimits components to handle empty specifications array safely - On self-hosted, specifications will be empty and not passed to backend, allowing default spec Co-authored-by: stnguyen90 <1477010+stnguyen90@users.noreply.github.com>
Done! Implemented the specification handling changes in commit a1deff6. Changes:
Verification:
Warning Firewall rules blocked me from connecting to one or more addresses (expand for details)I tried to connect to the following addresses, but was blocked by firewall rules:
If you need me to access, download, or install something from one of these locations, you can either:
|

Updates the Console to conditionally call
listSpecifications()only when running on Cloud. On self-hosted deployments, the specifications list will be empty, which means the specification parameter won't be passed when creating or updating functions or sites. This allows the backend to use the default (highest available) specification instead of the lowest.Changes Made
Layout Loaders
src/routes/(console)/project-[region]-[project]/functions/+layout.ts): AddedisCloudcheck to conditionally calllistSpecifications()- returns empty list on self-hostedsrc/routes/(console)/project-[region]-[project]/sites/site-[site]/settings/+page.ts): AddedisCloudcheck to conditionally calllistSpecifications()- returns empty list on self-hostedComponent Updates
Result
specificationparameter will be empty string or undefined when creating/updating, causing the backend to use the default (highest available) specificationTesting
The changes are minimal and focused, affecting only specification handling without altering other functionality.
Original prompt
💡 You can make Copilot smarter by setting up custom instructions, customizing its development environment and configuring Model Context Protocol (MCP) servers. Learn more Copilot coding agent tips in the docs.