Skip to content

fix(events): Log deprecated events as debug, not info - #37816

Merged
ChristophWurst merged 1 commit into
masterfrom
fix/events/info-to-debug-log-legacy-events
Apr 24, 2023
Merged

fix(events): Log deprecated events as debug, not info#37816
ChristophWurst merged 1 commit into
masterfrom
fix/events/info-to-debug-log-legacy-events

Conversation

@ChristophWurst

Copy link
Copy Markdown
Member

Summary

There is nothing an admin can. So let's not log this with info level.

Checklist

Signed-off-by: Christoph Wurst <christoph@winzerhof-wurst.at>
@ChristophWurstChristophWurst added the 3. to review Waiting for reviews label Apr 19, 2023
@ChristophWurstChristophWurst added this to the Nextcloud 27 milestone Apr 19, 2023
@ChristophWurstChristophWurst self-assigned this Apr 19, 2023

@szaimenszaimen left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Great idea! :)

@szaimenszaimen added 4. to release Ready to be released and/or waiting for tests to finish and removed 3. to review Waiting for reviews labels Apr 20, 2023
@ChristophWurst
ChristophWurst merged commit dcde970 into masterApr 24, 2023
@ChristophWurst
ChristophWurst deleted the fix/events/info-to-debug-log-legacy-events branch April 24, 2023 09:14
@szaimen

Copy link
Copy Markdown
Contributor

backports?

@ChristophWurst

Copy link
Copy Markdown
MemberAuthor

Trigger them if you think they are save and necessary. I am hesitant with this type of change.

@szaimen

Copy link
Copy Markdown
Contributor

All right, fine for me to leave it like this for now 👍

Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Labels

4. to releaseReady to be released and/or waiting for tests to finish

Projects

Archived in project

Development

Successfully merging this pull request may close these issues.

3 participants

@ChristophWurst@szaimen@Chartman123