8000 Fix media item preview in the editor by jessedobbelaere · Pull Request #3450 · forkcms/forkcms · GitHub
[go: up one dir, main page]

Skip to content

Fix media item preview in the editor#3450

Merged
carakas merged 1 commit intoforkcms:masterfrom
jessedobbelaere:fix-media-item-preview-part-2
Aug 30, 2021
Merged

Fix media item preview in the editor#3450
carakas merged 1 commit intoforkcms:masterfrom
jessedobbelaere:fix-media-item-preview-part-2

Conversation

@jessedobbelaere
Copy link
Member
@jessedobbelaere jessedobbelaere commented Aug 25, 2021

Type

  • Critical bugfix

Resolves the following issues

image

Pull request description

See similar PR: #3434

@codecov
Copy link
codecov bot commented Aug 25, 2021

Codecov Report

Merging #3450 (5654e12) into master (897d98d) will decrease coverage by 0.00%.
The diff coverage is 0.00%.

Impacted file tree graph

@@             Coverage Diff              @@
##             master    #3450      +/-   ##
============================================
- Coverage     27.88%   27.88%   -0.01%     
  Complexity     8145     8145              
============================================
  Files           575      575              
  Lines         30689    30690       +1     
============================================
  Hits           8558     8558              
- Misses        22131    22132       +1     
Flag Coverage Δ
functional 23.79% <0.00%> (-0.01%) ⬇️
installer 3.84% <0.00%> (-0.01%) ⬇️
unit 7.63% <0.00%> (-0.01%) ⬇️

Flags with carried forward coverage won't be shown. Click here to find out more.

Impacted Files Coverage Δ
...ry/Domain/MediaItem/MediaItemSelectionDataGrid.php 0.00% <0.00%> (ø)

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 897d98d...5654e12. Read the comment docs.

@carakas carakas merged commit a4734ad into forkcms:master Aug 30, 2021
@carakas carakas added this to the 5.11.0 milestone Aug 30, 2021
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.

2 participants

0