Skip to content

Use PNG icon on activity email#344

Merged
Ivansss merged 1 commit into
masterfrom
png-icon-in-emails
Jul 3, 2017
Merged

Use PNG icon on activity email#344
Ivansss merged 1 commit into
masterfrom
png-icon-in-emails

Conversation

@nickvergessen

Copy link
Copy Markdown
Member

Signed-off-by: Joas Schilling <coding@schilljs.com>
@nickvergessen nickvergessen added this to the 3.0 (Nextcloud 13) milestone Jun 20, 2017
@nickvergessen
nickvergessen requested a review from Ivansss June 20, 2017 13:39
@codecov

codecov Bot commented Jun 20, 2017

Copy link
Copy Markdown

Codecov Report

Merging #344 into master will decrease coverage by 0.03%.
The diff coverage is 0%.

Impacted file tree graph

@@             Coverage Diff              @@
##             master     #344      +/-   ##
============================================
- Coverage     15.62%   15.58%   -0.04%     
- Complexity      274      276       +2     
============================================
  Files            18       18              
  Lines          1293     1296       +3     
============================================
  Hits            202      202              
- Misses         1091     1094       +3
Impacted Files Coverage Δ Complexity Δ
lib/Activity/Provider.php 71.23% <0%> (-3.06%) 20 <0> (+2)

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 230a987...0a98d7d. Read the comment docs.

@Ivansss
Ivansss merged commit 7d919c4 into master Jul 3, 2017
@Ivansss
Ivansss deleted the png-icon-in-emails branch July 3, 2017 08:06
@MorrisJobke

Copy link
Copy Markdown
Member

@nickvergessen Mind to do the backport?

@MorrisJobke

Copy link
Copy Markdown
Member

@nickvergessen Ping on the backport

@nickvergessen

Copy link
Copy Markdown
Member Author

Beauty comes with 3.0 🙊

marcoambrosini pushed a commit that referenced this pull request Oct 9, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants