-
Notifications
You must be signed in to change notification settings - Fork 2.1k
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
Set mail_send via old share API #35136
Conversation
release-10.2.0: #35137 |
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.
👍 makes sense, thanks
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.
👍
Codecov Report
@@ Coverage Diff @@
## master #35136 +/- ##
============================================
+ Coverage 65.47% 65.48% +0.01%
+ Complexity 18636 18634 -2
============================================
Files 1217 1217
Lines 70545 70534 -11
Branches 1296 1296
============================================
Hits 46192 46192
+ Misses 23976 23965 -11
Partials 377 377
Continue to review full report at Codecov.
|
1 similar comment
Codecov Report
@@ Coverage Diff @@
## master #35136 +/- ##
============================================
+ Coverage 65.47% 65.48% +0.01%
+ Complexity 18636 18634 -2
============================================
Files 1217 1217
Lines 70545 70534 -11
Branches 1296 1296
============================================
Hits 46192 46192
+ Misses 23976 23965 -11
Partials 377 377
Continue to review full report at Codecov.
|
Description
Partially reverts 129d5e1
Related Issue
Motivation and Context
It's possible to notify group about share multiple times
How Has This Been Tested?
In accordance with #35089
Types of changes
Checklist:
Open tasks: