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

Feat: list relationships cmd #10898

Draft
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

Aijeyomah
Copy link
Contributor

@Aijeyomah Aijeyomah commented May 8, 2024

Notes for Reviewers

This PR fixes #9844

  • List relationships experimental conmand

Signed commits

  • Yes, I signed my commits.

Signed-off-by: Eti Ijeoma <53005190+Aijeyomah@users.noreply.github.com>
@github-actions github-actions bot added the component/mesheryctl CLI for Meshery label May 8, 2024
@Aijeyomah Aijeyomah changed the title Feat: list relationshipscmd Feat: list relationships cmd May 8, 2024
Copy link

github-actions bot commented May 8, 2024

}

relationshipsResponse := &models.MeshmodelRelationshipsAPIResponse{}
err = json.Unmarshal(data, relationshipsResponse)
Copy link
Contributor

Choose a reason for hiding this comment

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

Did you ran the command ?
Here it is giving error for me.

return err
}

header := []string{"Entity Details", "Type"}
Copy link
Contributor

Choose a reason for hiding this comment

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

Please refer here to see the columns we need to display.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Please refer here to see the columns we need to display.

Alright, thanks

return nil
}

if cmd.Flags().Changed("page") {
Copy link
Contributor

@Jougan-0 Jougan-0 May 14, 2024

Choose a reason for hiding this comment

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

For me hitting enter causes issue .

Copy link
Contributor Author

Choose a reason for hiding this comment

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

For me hitting enter causes issue .

Sorry about that, I still in draft because I was waiting for the interactive pagination to be merged and it was yesterday. I'm working on completing it today

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

Successfully merging this pull request may close these issues.

[mesheryctl] New command exp relationship
2 participants