Closed alysivji closed 4 years ago
Merging #317 into master will decrease coverage by
0.01%
. The diff coverage is92.45%
.
@@ Coverage Diff @@
## master #317 +/- ##
==========================================
- Coverage 92.35% 92.33% -0.02%
==========================================
Files 70 71 +1
Lines 2315 2362 +47
==========================================
+ Hits 2138 2181 +43
- Misses 177 181 +4
Impacted Files | Coverage Δ | |
---|---|---|
busy_beaver/common/wrappers/aws_s3.py | 89.18% <89.18%> (ø) |
|
busy_beaver/apps/web/forms.py | 100.00% <100.00%> (ø) |
|
busy_beaver/apps/web/views.py | 73.89% <100.00%> (+0.12%) |
:arrow_up: |
busy_beaver/clients.py | 100.00% <100.00%> (ø) |
|
busy_beaver/common/wrappers/__init__.py | 100.00% <100.00%> (ø) |
|
busy_beaver/config.py | 100.00% <100.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 49c75c7...2c9dca9. Read the comment docs.
Closes #295
What does this do
Instead of having users enter in a URL, we now allow users to upload logos to S3 like blob store. We now serve images from our Digital Ocean CDN.
Why are we doing this
Improve user experience.
How should this be tested
Added localstack to project in order to provide an S3-like store for development / CI.
Migrations
n/a
Dependencies
Callouts
n/a