0chain / blobber

A storage provider (blobber) interface to the blockchain and consumers of storage.
Other
19 stars 23 forks source link

process thumbnail before save #1391

Closed Hitenjain14 closed 6 months ago

Hitenjain14 commented 6 months ago

Changes

Fixes

Tests

Tasks to complete before merging PR:

Associated PRs (Link as appropriate):

codecov-commenter commented 6 months ago

Codecov Report

Attention: Patch coverage is 0% with 25 lines in your changes are missing coverage. Please review.

Project coverage is 15.00%. Comparing base (c8f4be5) to head (249501b).

Files Patch % Lines
...ain.net/blobbercore/handler/file_command_upload.go 0.00% 10 Missing and 3 partials :warning:
...ain.net/blobbercore/handler/file_command_update.go 0.00% 12 Missing :warning:

:exclamation: Your organization needs to install the Codecov GitHub app to enable full functionality.

Additional details and impacted files ```diff @@ Coverage Diff @@ ## sprint-1.13 #1391 +/- ## =============================================== - Coverage 15.01% 15.00% -0.01% =============================================== Files 178 178 Lines 19890 19898 +8 =============================================== Hits 2986 2986 - Misses 16535 16542 +7 - Partials 369 370 +1 ``` | [Flag](https://app.codecov.io/gh/0chain/blobber/pull/1391/flags?src=pr&el=flags&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=0chain) | Coverage Δ | | |---|---|---| | [Unit-Tests](https://app.codecov.io/gh/0chain/blobber/pull/1391/flags?src=pr&el=flag&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=0chain) | `15.00% <0.00%> (-0.01%)` | :arrow_down: | Flags with carried forward coverage won't be shown. [Click here](https://docs.codecov.io/docs/carryforward-flags?utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=0chain#carryforward-flags-in-the-pull-request-comment) to find out more.

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.