Skip to content

Conversation

@mkaz
Copy link
Member

@mkaz mkaz commented Jul 31, 2017

A little bit of cleanup - the slimImageObjects method removes unneeded attributes from the image objects. This method was previous done within gallery but was refactored and moved into the MediaUploadButton so no longer needed here. This change removes the method.

See #1820

@mkaz mkaz requested a review from westonruter July 31, 2017 21:43
@codecov
Copy link

codecov bot commented Jul 31, 2017

Codecov Report

Merging #2110 into master will decrease coverage by <.01%.
The diff coverage is 0%.

Impacted file tree graph

@@            Coverage Diff            @@
##           master   #2110      +/-   ##
=========================================
- Coverage   23.11%   23.1%   -0.01%     
=========================================
  Files         141     141              
  Lines        4405    4402       -3     
  Branches      747     747              
=========================================
- Hits         1018    1017       -1     
+ Misses       2856    2854       -2     
  Partials      531     531
Impacted Files Coverage Δ
blocks/library/gallery/index.js 25% <0%> (-0.72%) ⬇️

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 c2a0e5c...50907cd. Read the comment docs.

Copy link
Member

@aduth aduth left a comment

Choose a reason for hiding this comment

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

Needs rebase, but looks good.

@mkaz mkaz force-pushed the fix/gallery-slimimages branch from 9ea70d0 to 50907cd Compare August 4, 2017 16:21
@mkaz mkaz merged commit cd972a2 into master Aug 4, 2017
@mkaz mkaz deleted the fix/gallery-slimimages branch August 4, 2017 16:35
@isabellachen isabellachen mentioned this pull request Apr 1, 2020
3 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants