-
-
Notifications
You must be signed in to change notification settings - Fork 2.3k
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
[Streams] Only send message about missing client secret once #3901
Conversation
… last notify_owner
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Works great, I only wonder if notified_owner
is the best name here - there could be more situations where we would want to notify owner in future and it could be confusing if there were notified_owner
but also notified_owner_about_something
.
I'm approving this anyway, but if you want to change that name, feel free to.
There's also one string suggestion but it's not related to the actual change so it can be ignored too.
wait no, dont merge i agree with you, and to avoid a migration later lets do it now |
Co-authored-by: jack1142 <6032823+jack1142@users.noreply.github.com>
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks good to me 👍
…
Type
Description of the changes
fixes #3587