-
Notifications
You must be signed in to change notification settings - Fork 113
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
[Schema] Added publish modal schema #354
Conversation
Signed-off-by: yash sharma <[email protected]>
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 Technology enum is wrong.
schemas/publish/publishModal.json
Outdated
"items": { | ||
"enum": [ | ||
"istio", | ||
"linkerd" |
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.
What is this?
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.
This enum will be taken care of in Meshery UI because we fetch meshmdal names from API I added two items here in case to not pass empty enum
Co-authored-by: Lee Calcote <[email protected]> Signed-off-by: Yash sharma <[email protected]>
Co-authored-by: Lee Calcote <[email protected]> Signed-off-by: Yash sharma <[email protected]>
Co-authored-by: Lee Calcote <[email protected]> Signed-off-by: Yash sharma <[email protected]>
Co-authored-by: Lee Calcote <[email protected]> Signed-off-by: Yash sharma <[email protected]>
Signed-off-by: yash sharma <[email protected]>
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'm suggesting a change to the default technologies.
Signed-off-by: Lee Calcote <[email protected]>
Description
This PR fixes #
Notes for Reviewers
Signed commits