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

DXCDT-318: Fix log streams error message #598

Merged
merged 14 commits into from
Jan 11, 2023

Conversation

willvedd
Copy link
Contributor

@willvedd willvedd commented Jan 11, 2023

🔧 Changes

When trying to update log streams via auth0 logs streams update, if no log streams exist on your tenant, you're prompted with the message "there are currently no roles". This is obviously an error due to a likely copy and paste from the update roles command. This PR issues a correct error in this instance and pulls the original error out of the global package scope to prevent any other similar instances from occuring.

📝 Checklist

  • All new/changed/fixed functionality is covered by tests (or N/A)
  • I have added documentation for all new/changed functionality (or N/A)

@willvedd willvedd requested a review from a team as a code owner January 11, 2023 17:01
@sergiught sergiught merged commit d1ab6f1 into v1 Jan 11, 2023
@sergiught sergiught deleted the DXCDT-318-fix-no-log-streams-error branch January 11, 2023 17:25
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