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

improvement(cloud): log 'not logged in' msg at info level for community #5553

Merged
merged 1 commit into from
Dec 13, 2023

Conversation

eysi09
Copy link
Collaborator

@eysi09 eysi09 commented Dec 12, 2023

What this PR does / why we need it:

A previous commit (see #5538) logged this at the warning level which is relevant for the commercial edition but the community. This fixes that.

Which issue(s) this PR fixes:

Fixes #

Special notes for your reviewer:

@eysi09 eysi09 requested a review from vvagaytsev December 12, 2023 18:08
@eysi09 eysi09 enabled auto-merge December 12, 2023 19:57
core/src/garden.ts Outdated Show resolved Hide resolved
A previous commit logged this at the warning level which is relevant for
the commercial edition but the community. This fixes that.
@eysi09 eysi09 force-pushed the log-dashboard-warning-at-info-level branch from 1367ff1 to 8143b73 Compare December 13, 2023 09:46
Copy link
Collaborator

@vvagaytsev vvagaytsev left a comment

Choose a reason for hiding this comment

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

Thank you! ✅

@eysi09 eysi09 added this pull request to the merge queue Dec 13, 2023
Merged via the queue into main with commit cad5060 Dec 13, 2023
45 checks passed
@eysi09 eysi09 deleted the log-dashboard-warning-at-info-level branch December 13, 2023 11:06
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