Closed cyp3rius closed 2 years ago
Task linked: CU-20rfmhn Settings page
Merging #109 (320d36c) into master (eba4b70) will increase coverage by
3.36%
. The diff coverage is47.76%
.
@@ Coverage Diff @@
## master #109 +/- ##
==========================================
+ Coverage 23.11% 26.48% +3.36%
==========================================
Files 11 11
Lines 359 404 +45
Branches 119 128 +9
==========================================
+ Hits 83 107 +24
- Misses 218 238 +20
- Partials 58 59 +1
Flag | Coverage Δ | |
---|---|---|
unittest | 26.48% <47.76%> (+3.36%) |
:arrow_up: |
Flags with carried forward coverage won't be shown. Click here to find out more.
Impacted Files | Coverage Δ | |
---|---|---|
server/config/index.js | 100.00% <ø> (ø) |
|
server/services/admin.js | 6.45% <0.00%> (-1.89%) |
:arrow_down: |
server/services/client.js | 6.66% <11.11%> (-0.07%) |
:arrow_down: |
server/services/common.js | 37.96% <92.30%> (+11.58%) |
:arrow_up: |
server/services/utils/functions.js | 46.55% <100.00%> (+6.92%) |
:arrow_up: |
server/utils/constants.js | 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 eba4b70...320d36c. Read the comment docs.
Ticket
CU-20rfmhn
Summary
What does this PR do/solve?
Introduction of Settings page to dynamically configure the plugin.
Test Plan
Settings -> Comments plugin -> Configuration