-
Notifications
You must be signed in to change notification settings - Fork 101
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
Update schema for import #595
Conversation
Signed-off-by: Jougan-0 <[email protected]>
Signed-off-by: Jougan-0 <[email protected]>
"Analytics", | ||
"App Definition and Development", | ||
"Cloud Native Network", | ||
"Cloud Native Storage", | ||
"Database", | ||
"Machine Learning", | ||
"Observability and Analysis", | ||
"Orchestration & Management", | ||
"Platform", | ||
"Provisioning", | ||
"Runtime", | ||
"Security & Compliance", | ||
"Serverless", | ||
"Tools", | ||
"Uncategorized" |
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.
These should be fetched dynamically, look for the IMport design model (source type field) for reference on dynamically fetching.
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.
@MUzairS15 We used enum here to 🤔
"enum": [ |
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.
Then it must be some other schema, check catalog publish
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.
I'll make them dynamically after today's event.
}, | ||
"subCategory": { | ||
"type": "string", | ||
"enum": [ |
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.
same here
Description
This PR fixes #
Notes for Reviewers
Signed commits