Skip to content

Fix: Resize error name wasn't being displayed properly#2356

Merged
dpwatrous merged 1 commit intomasterfrom
dpwatrous/resize-error-key
Aug 3, 2021
Merged

Fix: Resize error name wasn't being displayed properly#2356
dpwatrous merged 1 commit intomasterfrom
dpwatrous/resize-error-key

Conversation

@dpwatrous
Copy link
Member

@dpwatrous dpwatrous commented Jul 30, 2021

The ResizeError details object has a format of { name: string, value: string }, not { key: string, value: string }, which the PoolErrorDisplayComponent was using.

@codecov
Copy link

codecov bot commented Jul 30, 2021

Codecov Report

Merging #2356 (3fc08f4) into master (84f940f) will increase coverage by 0.00%.
The diff coverage is n/a.

Impacted file tree graph

@@           Coverage Diff           @@
##           master    #2356   +/-   ##
=======================================
  Coverage   65.23%   65.24%           
=======================================
  Files         903      903           
  Lines       26024    26024           
  Branches     5115     5115           
=======================================
+ Hits        16977    16979    +2     
+ Misses       9047     9045    -2     
Impacted Files Coverage Δ
...ails/error-display/pool-error-display.component.ts 65.38% <0.00%> (+7.69%) ⬆️

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 84f940f...3fc08f4. Read the comment docs.

@dpwatrous dpwatrous merged commit 88bff28 into master Aug 3, 2021
@dpwatrous dpwatrous deleted the dpwatrous/resize-error-key branch August 3, 2021 17:50
@cRui861
Copy link
Member

cRui861 commented Aug 5, 2021

Fixed #2361

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