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.

Sign up for GitHub

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

Jump to bottom

ockam status should provide more user friendly information when a user is *not* enrolled #6064

Closed
nazmulidris opened this issue Sep 24, 2023 · 2 comments · Fixed by #6196
Closed

ockam status should provide more user friendly information when a user is *not* enrolled #6064

nazmulidris opened this issue Sep 24, 2023 · 2 comments · Fixed by #6196
Assignees
@rjtch
Labels
Component: Command good first issue hacktoberfest Apply to issues you want contributors to help with help wanted Implementation: Rust Type: User Experience

Comments

@nazmulidris
Copy link
Contributor

nazmulidris commented Sep 24, 2023 •
edited by mrinalwadhwa
Loading

Current behavior

When a user is not enrolled (they have not run ockam enroll yet), and they run ockam status they currently get an error message of the empty local state. This information is not very useful. Here's a screenshot of what this looks like.

Image

To install Ockam command from a prebuilt binary or from source, please take a look at this README.md.

Desired behavior

If a user is not enrolled, and they run ockam status, they should see a helpful message such as “No identities are found. Consider running ockam enroll to enroll an identity.” instead of just an error message.

We love helping new contributors! ❤️
If you have questions or need help as you explore, please join us on Discord. If you're looking for other issues to contribute to, please checkout our good first issues.
@rjtch rjtch mentioned this issue Sep 27, 2023
Closed
@rjtch
Copy link
Contributor

rjtch commented Sep 27, 2023

Hey @nazmulidris i want to work on this task

@nazmulidris
Copy link
Contributor Author

@rjtch Excellent, I just assigned it to you! 👍🏽

@adrianbenavides adrianbenavides mentioned this issue Oct 3, 2023
Merged
5 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@rjtch rjtch

Labels
Component: Command good first issue hacktoberfest Apply to issues you want contributors to help with help wanted Implementation: Rust Type: User Experience
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

changed status error messages rjtch/ockam
2 participants
@nazmulidris @rjtch