Skip to content
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

support generation of model from openapi schemas #582

Merged
merged 13 commits into from
Sep 30, 2024

Conversation

MUzairS15
Copy link

Description

This PR fixes #

Notes for Reviewers

Signed commits

  • Yes, I signed my commits.

MUzairS15 added 2 commits August 27, 2024 19:34
Copy link
Member

@leecalcote leecalcote left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@MUzairS15 build checks are failing...

Signed-off-by: MUzairS15 <[email protected]>
@MUzairS15
Copy link
Author

ensure correct version is passed.

"github.com/meshery/schemas/models/v1beta1/model"
)

func GenerateFromOpenAPI(resource string) ([]component.ComponentDefinition, error) {
Copy link
Author

@MUzairS15 MUzairS15 Sep 13, 2024

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

pass the "pkg" that contains info regarding model to generate.

Copy link
Member

@leecalcote leecalcote left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Still needed?

@MUzairS15
Copy link
Author

MUzairS15 commented Sep 25, 2024

// @Jougan-0

MUzairS15 added 4 commits September 25, 2024 14:09
@MUzairS15 MUzairS15 requested a review from Jougan-0 September 26, 2024 20:08
MUzairS15 added 3 commits September 30, 2024 15:31
Signed-off-by: MUzairS15 <[email protected]>
Signed-off-by: MUzairS15 <[email protected]>
Signed-off-by: MUzairS15 <[email protected]>
@MUzairS15 MUzairS15 merged commit 14ba2d8 into meshery:master Sep 30, 2024
3 of 4 checks passed
Name: pkg.GetName(),
DisplayName: manifests.FormatToReadableString(pkg.GetName()),
Category: category.CategoryDefinition{
Name: "Orchestration & Management",
Copy link
Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Update

DisplayName: manifests.FormatToReadableString(kind),
Model: model.ModelDefinition{
SchemaVersion: v1beta1.ModelSchemaVersion,
Version: "v1.0.0",
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Why is this still hardcoded?


c := component.ComponentDefinition{
SchemaVersion: v1beta1.ComponentSchemaVersion,
Version: "v1.0.0",
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Hardcoding appropriate?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants