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

[stable12] use the email address configured in Nextcloud as sender instead of the users email address #5303

Merged
merged 1 commit into from Jun 13, 2017

Conversation

Projects
None yet
3 participants
@schiessle
Member

schiessle commented Jun 8, 2017

backport of #5302

use the email address configured in Nextcloud as sender instead of th…
…e users email address

Signed-off-by: Bjoern Schiessle <bjoern@schiessle.org>
@codecov

This comment has been minimized.

Show comment
Hide comment
@codecov

codecov bot Jun 8, 2017

Codecov Report

Merging #5303 into stable12 will decrease coverage by 0.01%.
The diff coverage is 0%.

@@              Coverage Diff               @@
##             stable12    #5303      +/-   ##
==============================================
- Coverage       54.18%   54.17%   -0.02%     
  Complexity      22294    22294              
==============================================
  Files            1380     1380              
  Lines           85347    85355       +8     
  Branches         1322     1322              
==============================================
- Hits            46244    46237       -7     
- Misses          39103    39118      +15
Impacted Files Coverage Δ Complexity Δ
apps/sharebymail/lib/ShareByMailProvider.php 58.91% <0%> (-0.96%) 84 <0> (ø)
apps/comments/lib/EventHandler.php 79.16% <0%> (-8.34%) 7% <0%> (ø)
core/js/js.js 61.21% <0%> (-0.57%) 0% <0%> (ø)

codecov bot commented Jun 8, 2017

Codecov Report

Merging #5303 into stable12 will decrease coverage by 0.01%.
The diff coverage is 0%.

@@              Coverage Diff               @@
##             stable12    #5303      +/-   ##
==============================================
- Coverage       54.18%   54.17%   -0.02%     
  Complexity      22294    22294              
==============================================
  Files            1380     1380              
  Lines           85347    85355       +8     
  Branches         1322     1322              
==============================================
- Hits            46244    46237       -7     
- Misses          39103    39118      +15
Impacted Files Coverage Δ Complexity Δ
apps/sharebymail/lib/ShareByMailProvider.php 58.91% <0%> (-0.96%) 84 <0> (ø)
apps/comments/lib/EventHandler.php 79.16% <0%> (-8.34%) 7% <0%> (ø)
core/js/js.js 61.21% <0%> (-0.57%) 0% <0%> (ø)

@nickvergessen nickvergessen merged commit f6264ad into stable12 Jun 13, 2017

1 of 3 checks passed

codecov/patch 0% of diff hit (target 54.18%)
Details
codecov/project 54.17% (-0.02%) compared to d00c588
Details
continuous-integration/drone/pr the build was successful
Details

@nickvergessen nickvergessen deleted the fix_5047-nc12 branch Jun 13, 2017

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