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

cli: do not panic on missing/invalid creds #93

Merged
merged 2 commits into from
Oct 10, 2023
Merged

cli: do not panic on missing/invalid creds #93

merged 2 commits into from
Oct 10, 2023

Conversation

pgavlin
Copy link
Member

@pgavlin pgavlin commented Oct 10, 2023

  • If creds are missing, return a friendly message instructing the user to log in
  • If the active account is not a cloud account, return a message indicating that the backend is not supported

Fixes #89.

- If creds are missing, return a friendly message instructing the user
  to log in
- If the active account is not a cloud account, return a message
  indicating that the backend is not supported

Fixes #89.
@pgavlin pgavlin merged commit 1312afb into main Oct 10, 2023
@pgavlin pgavlin deleted the pgavlin/login branch October 10, 2023 16:57
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.

Panic when listing environments when not logged in
3 participants