Conversation
Test results 13 files 13 suites 25m 10s ⏱️ Results for commit 92c6b57. ♻️ This comment has been updated with latest results. |
Codecov Report✅ All modified and coverable lines are covered by tests. Additional details and impacted files@@ Coverage Diff @@
## master #3888 +/- ##
==========================================
+ Coverage 63.74% 63.83% +0.08%
==========================================
Files 624 624
Lines 46154 46175 +21
Branches 43 43
==========================================
+ Hits 29421 29474 +53
+ Misses 16723 16691 -32
Partials 10 10 ☔ View full report in Codecov by Sentry. 🚀 New features to boost your workflow:
|
c614bfe to
efbbc5a
Compare
johannaengland
left a comment
There was a problem hiding this comment.
I noticed that for other attributes (like parent) that a line is only added if the attribute is not null, so to follow that pattern I would only show this line if a location has aliases
Funnily enough in rooms that is not the case - in that case you should create a follow up issue with a discussion tag where we should talk about if that difference in behavior is wanted and if not, which approach to follow - always show all lines or only should the lines where the content is not empty |
done here #3893 |
92c6b57 to
de1044f
Compare
|



Scope and purpose
Fixes #3841.
before:

after:

with aliases
without
(same as before)
Contributor Checklist
Every pull request should have this checklist filled out, no matter how small it is.
More information about contributing to NAV can be found in the
Hacker's guide to NAV.
<major>.<minor>.x). For a new feature or other additions, it should be based onmaster.