Skip to content

Conversation

@ChristophWurst
Copy link
Member

Summary

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

Checklist

Copy link
Contributor

@szaimen szaimen left a comment

Choose a reason for hiding this comment

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

Great idea! :)

@szaimen szaimen 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 master Apr 24, 2023
@ChristophWurst ChristophWurst deleted the fix/events/info-to-debug-log-legacy-events branch April 24, 2023 09:14
@szaimen
Copy link
Contributor

szaimen commented Apr 24, 2023

backports?

@ChristophWurst
Copy link
Member Author

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

@szaimen
Copy link
Contributor

szaimen commented Apr 24, 2023

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

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

Labels

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

Projects

Development

Successfully merging this pull request may close these issues.

4 participants