Codecov Report

Merging #1754 (7ca6f1b) into develop (f38f6be) will increase coverage by 0.10%.
The diff coverage is n/a.

@@             Coverage Diff             @@
##           develop    #1754      +/-   ##
===========================================
+ Coverage    83.34%   83.43%   +0.10%     
===========================================
  Files          468      468              
  Lines        16304    16301       -3     
  Branches      2431     2430       -1     
===========================================
+ Hits         13587    13600      +13     
+ Misses        2050     2034      -16     
  Partials       667      667              

I'm not sure this is the right approach, because when we currently validate data we drop base64 property here. resourceId property also relies on the drawable content, the same way as base64 (essentially they are using the same input, correct me if I'm wrong), so since input for base64 is considered as inconsistent, we should also remove resourceId from the validation instead of adding it to the json. WDYT?

I'm not sure this is the right approach, because when we currently validate data we drop base64 property here. resourceId property also relies on the drawable content, the same way as base64 (essentially they are using the same input, correct me if I'm wrong), so since input for base64 is considered as inconsistent, we should also remove resourceId from the validation instead of adding it to the json. WDYT?

Yes, I recall now. Marius had failures on CI because of the variability - good catch. I've changed to remove the resourceId field