-
Notifications
You must be signed in to change notification settings - Fork 2
feat: Add project creation UI with configuration support #25
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: yerek
Are you sure you want to change the base?
Conversation
Co-authored-by: Hye-Dev <[email protected]>
Co-authored-by: Hye-Dev <[email protected]>
Co-authored-by: Hye-Dev <[email protected]>
Co-authored-by: Hye-Dev <[email protected]>
Hye-Dev
left a comment
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.
The add project button should be in the dropdown. Not a plus icon in the header.
Don't toast on project creation success
Render project options correctly
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.
- Don't toast on project creation success
- Configurable options should render properly. Not just inputs. useConfigurableOptions() should yield something like the file attached: example.json
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.
Done in commit 2726407:
- ✅ Removed success toast - the dialog now closes silently on successful creation
- ✅ Configuration options now render properly based on the metadata structure:
- Boolean fields (
type: "bool") render as checkboxes - Integer fields (
type: "integer") render as number inputs - String fields render as text inputs
- Options are grouped by category (features, limits, restricted, etc.) in expandable accordions
- Parses the nested structure from the example.json format with
keysarrays containing config objects
- Boolean fields (
|
@copilot please make the changes requested |
Co-authored-by: Hye-Dev <[email protected]>
Implemented in commit 2726407:
|
Implements project creation flow per Incus API spec, including initial configuration from
/1.0/metadata/configurationendpoint.Changes
API Layer (
app/(main)/_lib/projects.ts)createProject()function callingPOST /1.0/projectsCreateProjectRequestinterface for name, description, and config mapUI Component (
app/(main)/_components/createProject.tsx)^[a-z][a-z0-9-]*[a-z0-9]$(1-63 chars)type: "bool") render as checkboxestype: "integer") render as number inputsHeader Integration (
app/(main)/_components/header.tsx)Supporting Component (
app/_components/textarea.tsx)Configuration keys dynamically populated from
useConfigurableOptions().data.configs.project, properly parsing the nested metadata structure with categories and field types.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.