Closed ddabble closed 2 years ago
Merging #569 (7f0540c) into dev (6333885) will decrease coverage by
0.70%
. The diff coverage is46.66%
.
@@ Coverage Diff @@
## dev #569 +/- ##
==========================================
- Coverage 88.46% 87.76% -0.71%
==========================================
Files 132 133 +1
Lines 5514 5572 +58
==========================================
+ Hits 4878 4890 +12
- Misses 636 682 +46
Impacted Files | Coverage Δ | |
---|---|---|
internal/models.py | 93.28% <ø> (ø) |
|
makerspace/models.py | 100.00% <ø> (ø) |
|
news/models.py | 98.69% <ø> (ø) |
|
util/ckeditor_utils.py | 0.00% <0.00%> (ø) |
|
util/templatetags/string_tags.py | 100.00% <ø> (ø) |
|
web/hosts.py | 100.00% <ø> (ø) |
|
util/locale_utils.py | 82.75% <60.00%> (-8.36%) |
:arrow_down: |
make_queue/templatetags/reservation_extra.py | 94.20% <71.42%> (-1.32%) |
:arrow_down: |
checkin/models.py | 92.59% <100.00%> (+1.52%) |
:arrow_up: |
contentbox/views.py | 100.00% <100.00%> (ø) |
|
... and 11 more |
Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.
Proposed changes
Improvements
trans
template tag withtranslate
(46a81ff2162cb1ed98643d03e6677053a5e073c7)user
field ofProfile
non-nullable (4dc7dff842ad30dde2d12e009ba713f149cddd36)Fixes
Other changes
Areas to review closely
Ensure that all pages have a (sensibly) set title (through the
<title>
tag).Deployment notes
The base folder for all uploaded CKEditor files was changed from
ckeditor-upload
tockeditor_upload
, so the existing folder on the server should be manually renamed while deploying. All references to files in that folder (only in inserted images in rich text fields, like those of articles andContentBox
es) must also be updated - either manually on the website, or through the Django shell on the server.Checklist
(If any of the points are not relevant, mark them as checked)